Bug 1406047 - Schedule a composite in empty transactions that require it. r?jrmuizel draft
authorKartikaya Gupta <kgupta@mozilla.com>
Thu, 05 Oct 2017 11:42:41 -0400
changeset 675594 c72fa82e94d976d85c767d02682606030cd24d8e
parent 675508 53bbdaaa2b8c1819061be26101b075c081b23260
child 734648 7543084d579bd66868ab8623b1c342e0af1f8a79
push id83178
push userkgupta@mozilla.com
push dateThu, 05 Oct 2017 15:43:00 +0000
reviewersjrmuizel
bugs1406047
milestone58.0a1
Bug 1406047 - Schedule a composite in empty transactions that require it. r?jrmuizel MozReview-Commit-ID: 2e3auabQvtw
gfx/layers/wr/WebRenderLayerManager.cpp
--- a/gfx/layers/wr/WebRenderLayerManager.cpp
+++ b/gfx/layers/wr/WebRenderLayerManager.cpp
@@ -185,16 +185,19 @@ WebRenderLayerManager::EndEmptyTransacti
   // We also need to update canvases that might have changed, but this code
   // as-is causes crashes so comment it out for now.
   //for (auto iter = mLastCanvasDatas.Iter(); !iter.Done(); iter.Next()) {
   //  RefPtr<WebRenderCanvasData> canvasData = iter.Get()->GetKey();
   //  WebRenderCanvasRendererAsync* canvas = canvasData->GetCanvasRenderer();
   //  canvas->UpdateCompositableClient();
   //}
 
+  if (!(aFlags & EndTransactionFlags::END_NO_COMPOSITE)) {
+    ScheduleComposite();
+  }
   return true;
 }
 
 /*static*/ int32_t
 PopulateScrollData(WebRenderScrollData& aTarget, Layer* aLayer)
 {
   MOZ_ASSERT(aLayer);