Bug 1435282 - Document that function arguments are always null. r?sotaro draft
authorKartikaya Gupta <kgupta@mozilla.com>
Fri, 02 Feb 2018 10:48:25 -0500
changeset 750553 2d85fa4f8f9699cc1dc0ac5cc6c5f1ca5adf5a95
parent 750552 ef353d2c98b1e570c682bd9c0e1bc35f86290702
child 750554 dd5fce9853d01a8b422d0bfe187f336c431f323c
push id97707
push userkgupta@mozilla.com
push dateFri, 02 Feb 2018 17:00:40 +0000
reviewerssotaro
bugs1435282
milestone60.0a1
Bug 1435282 - Document that function arguments are always null. r?sotaro MozReview-Commit-ID: HOu4fshHSuX
gfx/layers/wr/WebRenderBridgeParent.cpp
--- a/gfx/layers/wr/WebRenderBridgeParent.cpp
+++ b/gfx/layers/wr/WebRenderBridgeParent.cpp
@@ -1194,16 +1194,21 @@ WebRenderBridgeParent::SampleAnimations(
       }
     }
   }
 }
 
 void
 WebRenderBridgeParent::CompositeToTarget(gfx::DrawTarget* aTarget, const gfx::IntRect* aRect)
 {
+  // The two arguments are part of the CompositorVsyncSchedulerOwner API but in
+  // this implementation they should never be non-null.
+  MOZ_ASSERT(aTarget == nullptr);
+  MOZ_ASSERT(aRect == nullptr);
+
   AUTO_PROFILER_TRACING("Paint", "CompositeToTraget");
   if (mPaused) {
     return;
   }
 
   if (!mForceRendering &&
       wr::RenderThread::Get()->TooManyPendingFrames(mApi->GetId())) {
     // Render thread is busy, try next time.