Get rid of SVGLayoutSupport::filtersForceContainerLayout

third_party/WebKit/Source/core/layout/svg/LayoutSVGRoot.cpp

 /*
  * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2007, 2008, 2009 Rob Buis <buis@kde.org>
  * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Google, Inc.
  * Copyright (C) Research In Motion Limited 2011. 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
  * License as published by the Free Software Foundation; either
@@ skipping 133 matching lines @@
     LayoutSize oldSize = size();
     updateLogicalWidth();
     updateLogicalHeight();
     buildLocalToBorderBoxTransform();
 
     SVGLayoutSupport::layoutResourcesIfNeeded(this);
 
     SVGSVGElement* svg = toSVGSVGElement(node());
     ASSERT(svg);
     m_isLayoutSizeChanged = needsLayout || (svg->hasRelativeLengths() && oldSize != size());
-    SVGLayoutSupport::layoutChildren(this, needsLayout || SVGLayoutSupport::filtersForceContainerLayout(this));
+    // If any of this root's children need to be laid out, and a filter is
+    // applied to it, we need to issue paint invalidations for all descendants.
+    bool forceLayoutOfChildren = needsLayout

[pdr., 2016/04/19 18:14:46]

This is a little confusing because the 2nd argument of layoutChildren is
"selfNeedsLayout". WDYT about renaming that to forceLayoutOfChildren or similar?
Could also push the selfNeedsLayout check into layoutChildren since all
callsites use that.

+        || (normalChildNeedsLayout() && SVGLayoutSupport::hasFilterResource(*this));
+    SVGLayoutSupport::layoutChildren(this, forceLayoutOfChildren);
 
     if (m_needsBoundariesOrTransformUpdate) {
         updateCachedBoundaries();
         m_needsBoundariesOrTransformUpdate = false;
     }
 
     m_overflow.clear();
     addVisualEffectOverflow();
 
     if (!shouldApplyViewportClip()) {
@@ skipping 239 matching lines @@
             updateHitTestResult(result, pointInBorderBox);
             if (result.addNodeToListBasedTestResult(node(), locationInContainer, boundsRect) == StopHitTesting)
                 return true;
         }
     }
 
     return false;
 }
 
 } // namespace blink