Bug 1615980 - Add more logs to RenderBufferTextureHost::Lock() r=nical

Differential Revision: https://phabricator.services.mozilla.com/D63044
mozilla-spidermonkey · Feb 17, 2020 · 7f7791c · 7f7791c
1 parent 0a5c1ea
commit 7f7791c
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/gfx/webrender_bindings/RenderBufferTextureHost.cpp b/gfx/webrender_bindings/RenderBufferTextureHost.cpp
@@ -52,6 +52,7 @@ wr::WrExternalImage RenderBufferTextureHost::Lock(
   if (!mLocked) {
     if (!GetBuffer()) {
       // We hit some problems to get the shmem.
+      gfxCriticalNote << "GetBuffer Failed";
       return InvalidToWrExternalImage();
     }
     if (mFormat != gfx::SurfaceFormat::YUV) {
@@ -61,11 +62,13 @@ wr::WrExternalImage RenderBufferTextureHost::Lock(
               mDescriptor.get_RGBDescriptor()),
           mSize, mFormat);
       if (NS_WARN_IF(!mSurface)) {
+        gfxCriticalNote << "DataSourceSurface is null";
         return InvalidToWrExternalImage();
       }
       if (NS_WARN_IF(!mSurface->Map(gfx::DataSourceSurface::MapType::READ_WRITE,
                                     &mMap))) {
         mSurface = nullptr;
+        gfxCriticalNote << "Failed to map Surface";
         return InvalidToWrExternalImage();
       }
     } else {
@@ -82,6 +85,7 @@ wr::WrExternalImage RenderBufferTextureHost::Lock(
           desc.cbCrStride(), desc.cbCrSize(), gfx::SurfaceFormat::A8);
       if (NS_WARN_IF(!mYSurface || !mCbSurface || !mCrSurface)) {
         mYSurface = mCbSurface = mCrSurface = nullptr;
+        gfxCriticalNote << "YCbCr Surface is null";
         return InvalidToWrExternalImage();
       }
       if (NS_WARN_IF(
@@ -92,6 +96,7 @@ wr::WrExternalImage RenderBufferTextureHost::Lock(
               !mCrSurface->Map(gfx::DataSourceSurface::MapType::READ_WRITE,
                                &mCrMap))) {
         mYSurface = mCbSurface = mCrSurface = nullptr;
+        gfxCriticalNote << "Failed to map YCbCr Surface";
         return InvalidToWrExternalImage();
       }
     }