From 26d69b10cf8850883af446b3b8e16807d4a75af3 Mon Sep 17 00:00:00 2001 From: Tetsuharu Ohzeki Date: Thu, 9 Sep 2021 15:46:20 +0000 Subject: [PATCH] Bug 1729817 - Remove unused WebRenderImageHost::PrintInfo(). r=gfx-reviewers,mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D125018 --- gfx/layers/composite/CompositableHost.h | 2 -- gfx/layers/composite/TextureHost.cpp | 25 ------------------------- gfx/layers/composite/TextureHost.h | 1 - gfx/layers/wr/WebRenderImageHost.cpp | 14 -------------- gfx/layers/wr/WebRenderImageHost.h | 2 -- 5 files changed, 44 deletions(-) diff --git a/gfx/layers/composite/CompositableHost.h b/gfx/layers/composite/CompositableHost.h index b4630837de51..968c3b8ea6ee 100644 --- a/gfx/layers/composite/CompositableHost.h +++ b/gfx/layers/composite/CompositableHost.h @@ -192,8 +192,6 @@ class CompositableHost { return nullptr; } - virtual void PrintInfo(std::stringstream& aStream, const char* aPrefix) = 0; - struct TimedTexture { CompositableTextureHostRef mTexture; TimeStamp mTimeStamp; diff --git a/gfx/layers/composite/TextureHost.cpp b/gfx/layers/composite/TextureHost.cpp index ff6a314cda39..60c191303018 100644 --- a/gfx/layers/composite/TextureHost.cpp +++ b/gfx/layers/composite/TextureHost.cpp @@ -479,31 +479,6 @@ void TextureHost::EnsureRenderTexture( CreateRenderTexture(mExternalImageId.ref()); } -void TextureHost::PrintInfo(std::stringstream& aStream, const char* aPrefix) { - aStream << aPrefix; - aStream << nsPrintfCString("%s (0x%p)", Name(), this).get(); - // Note: the TextureHost needs to be locked before it is safe to call - // GetSize() and GetFormat() on it. - if (Lock()) { - aStream << " [size=" << GetSize() << "]" - << " [format=" << GetFormat() << "]"; - Unlock(); - } - aStream << " [flags=" << mFlags << "]"; -#ifdef MOZ_DUMP_PAINTING - if (StaticPrefs::layers_dump_texture()) { - nsAutoCString pfx(aPrefix); - pfx += " "; - - aStream << "\n" << pfx.get() << "Surface: "; - RefPtr dSurf = GetAsSurface(); - if (dSurf) { - aStream << gfxUtils::GetAsLZ4Base64Str(dSurf).get(); - } - } -#endif -} - void TextureHost::Updated(const nsIntRegion* aRegion) { UpdatedInternal(aRegion); } diff --git a/gfx/layers/composite/TextureHost.h b/gfx/layers/composite/TextureHost.h index 88b74efb84c9..c58e2a65f389 100644 --- a/gfx/layers/composite/TextureHost.h +++ b/gfx/layers/composite/TextureHost.h @@ -606,7 +606,6 @@ class TextureHost : public AtomicRefCountedWithFinalize { virtual void ForgetBufferActor() {} virtual const char* Name() { return "TextureHost"; } - virtual void PrintInfo(std::stringstream& aStream, const char* aPrefix); /** * Indicates whether the TextureHost implementation is backed by an diff --git a/gfx/layers/wr/WebRenderImageHost.cpp b/gfx/layers/wr/WebRenderImageHost.cpp index 753aa1b322b6..94d0955a31c8 100644 --- a/gfx/layers/wr/WebRenderImageHost.cpp +++ b/gfx/layers/wr/WebRenderImageHost.cpp @@ -202,20 +202,6 @@ void WebRenderImageHost::SetTextureSourceProvider( CompositableHost::SetTextureSourceProvider(aProvider); } -void WebRenderImageHost::PrintInfo(std::stringstream& aStream, - const char* aPrefix) { - aStream << aPrefix; - aStream << nsPrintfCString("WebRenderImageHost (0x%p)", this).get(); - - nsAutoCString pfx(aPrefix); - pfx += " "; - for (const auto& img : Images()) { - aStream << "\n"; - img.mTextureHost->PrintInfo(aStream, pfx.get()); - aStream << " [picture-rect=" << img.mPictureRect << "]"; - } -} - void WebRenderImageHost::Dump(std::stringstream& aStream, const char* aPrefix, bool aDumpHtml) { for (const auto& img : Images()) { diff --git a/gfx/layers/wr/WebRenderImageHost.h b/gfx/layers/wr/WebRenderImageHost.h index f2fb463ff945..1b663308b77f 100644 --- a/gfx/layers/wr/WebRenderImageHost.h +++ b/gfx/layers/wr/WebRenderImageHost.h @@ -52,8 +52,6 @@ class WebRenderImageHost : public CompositableHost, public ImageComposite { gfx::IntSize GetImageSize() override; - void PrintInfo(std::stringstream& aStream, const char* aPrefix) override; - void Dump(std::stringstream& aStream, const char* aPrefix = "", bool aDumpHtml = false) override;