Bug 1264592 - When doing thumbnails for fennec, don't mark frames as painted because it can interfere with tests. r?mstange draft
authorKartikaya Gupta <kgupta@mozilla.com>
Fri, 06 May 2016 10:31:11 -0400
changeset 364381 8c096453f6c508b48715db8dcd02799e1830c743
parent 364352 19a1743ceb2e035e571012e88d25275ce627b925
child 520256 803c6c529490d985fcc3958d1a02ce04917774b0
push id17427
push userkgupta@mozilla.com
push dateFri, 06 May 2016 14:31:37 +0000
reviewersmstange
bugs1264592
milestone49.0a1
Bug 1264592 - When doing thumbnails for fennec, don't mark frames as painted because it can interfere with tests. r?mstange MozReview-Commit-ID: KOhZZV3aSLv
layout/base/FrameLayerBuilder.cpp
--- a/layout/base/FrameLayerBuilder.cpp
+++ b/layout/base/FrameLayerBuilder.cpp
@@ -5549,17 +5549,19 @@ FrameLayerBuilder::PaintItems(nsTArray<C
     }
 
     if (cdi->mInactiveLayerManager) {
       bool saved = aDrawTarget.GetPermitSubpixelAA();
       PaintInactiveLayer(aBuilder, cdi->mInactiveLayerManager, cdi->mItem, aContext, aRC);
       aDrawTarget.SetPermitSubpixelAA(saved);
     } else {
       nsIFrame* frame = cdi->mItem->Frame();
-      frame->AddStateBits(NS_FRAME_PAINTED_THEBES);
+      if (aBuilder->IsPaintingToWindow()) {
+        frame->AddStateBits(NS_FRAME_PAINTED_THEBES);
+      }
 #ifdef MOZ_DUMP_PAINTING
       if (gfxEnv::DumpPaintItems()) {
         DebugPaintItem(aDrawTarget, aPresContext, cdi->mItem, aBuilder);
       } else {
 #else
       {
 #endif
         cdi->mItem->Paint(aBuilder, aRC);