From 44e635d2e687c7b82102a729dfe35cd48d7e7fcc Mon Sep 17 00:00:00 2001 From: sotaro Date: Mon, 13 Feb 2023 21:46:54 +0000 Subject: [PATCH] Bug 1816053 - Add log to VideoBridgeParent::ActorDestroy() r=gfx-reviewers,lsalzman It would be helpful if VideoBridgeParent::ActorDestroy() has a log of AbnormalShutdown. Differential Revision: https://phabricator.services.mozilla.com/D169418 --- gfx/layers/ipc/CompositorBridgeChild.cpp | 3 ++- gfx/layers/ipc/VideoBridgeParent.cpp | 4 ++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/gfx/layers/ipc/CompositorBridgeChild.cpp b/gfx/layers/ipc/CompositorBridgeChild.cpp index 54bef1769ec6..ad227c611d85 100644 --- a/gfx/layers/ipc/CompositorBridgeChild.cpp +++ b/gfx/layers/ipc/CompositorBridgeChild.cpp @@ -314,7 +314,8 @@ void CompositorBridgeChild::ActorDestroy(ActorDestroyReason aWhy) { // If the parent side runs into a problem then the actor will be destroyed. // There is nothing we can do in the child side, here sets mCanSend as // false. - gfxCriticalNote << "Receive IPC close with reason=AbnormalShutdown"; + gfxCriticalNote << "CompositorBridgeChild receives IPC close with " + "reason=AbnormalShutdown"; } mCanSend = false; diff --git a/gfx/layers/ipc/VideoBridgeParent.cpp b/gfx/layers/ipc/VideoBridgeParent.cpp index 21968a067f84..44e0ccdbda4f 100644 --- a/gfx/layers/ipc/VideoBridgeParent.cpp +++ b/gfx/layers/ipc/VideoBridgeParent.cpp @@ -90,6 +90,10 @@ TextureHost* VideoBridgeParent::LookupTexture(uint64_t aSerial) { } void VideoBridgeParent::ActorDestroy(ActorDestroyReason aWhy) { + if (aWhy == AbnormalShutdown) { + gfxCriticalNote + << "VideoBridgeParent receives IPC close with reason=AbnormalShutdown"; + } // Can't alloc/dealloc shmems from now on. mClosed = true; }