Move RenderLayer repainting to repaint-after-layout framework.

Source/core/rendering/RenderLayerModelObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2010, 2012 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 130 matching lines @@
     RenderObject::styleDidChange(diff, oldStyle);
     updateFromStyle();
 
     LayerType type = layerTypeRequired();
     if (type != NoLayer) {
         if (!layer() && layerCreationAllowedForSubtree()) {
             if (s_wasFloating && isFloating())
                 setChildNeedsLayout();
             createLayer(type);
             if (parent() && !needsLayout() && containingBlock()) {
-                layer()->repainter().setRepaintStatus(NeedsFullRepaint);
+                if (RuntimeEnabledFeatures::repaintAfterLayoutEnabled())
+                    layer()->renderer()->setShouldDoFullRepaintAfterLayout(true);

[Julien - ping for review, 2014/03/04 18:22:10]

This seems overly broad as we could get a layer without visual impact (e.g. an
overflow only layer or a forced one). We should have a FIXME about removing this
crazy hammer and doing the right invalidation at RenderStyle::diff time.

[dsinclair, 2014/03/04 19:21:52]

Add a FIXME and crbug.com/349061.

+                else
+                    layer()->repainter().setRepaintStatus(NeedsFullRepaint);
                 // There is only one layer to update, it is not worth using |cachedOffset| since
                 // we are not sure the value will be used.
                 layer()->updateLayerPositions(0);
             }
         }
     } else if (layer() && layer()->parent()) {
         setHasTransform(false); // Either a transform wasn't specified or the object doesn't support transforms, so just null out the bit.
         setHasReflection(false);
         layer()->removeOnlyThisLayer(); // calls destroyLayer() which clears m_layer
         if (s_wasFloating && isFloating())
@@ skipping 52 matching lines @@
     return m_layer ? m_layer->hasCompositedLayerMapping() : false;
 }
 
 CompositedLayerMapping* RenderLayerModelObject::groupedMapping() const
 {
     return m_layer ? m_layer->groupedMapping() : 0;
 }
 
 } // namespace WebCore