Hoist transformToRootChanged() out of SVGLayoutSupport::layoutChildren

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

 /*
  * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
  *
  * 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
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 15 matching lines @@
 
 LayoutSVGHiddenContainer::LayoutSVGHiddenContainer(SVGElement* element)
     : LayoutSVGContainer(element)
 {
 }
 
 void LayoutSVGHiddenContainer::layout()
 {
     ASSERT(needsLayout());
     LayoutAnalyzer::Scope analyzer(*this);
-    SVGLayoutSupport::layoutChildren(this, selfNeedsLayout());
+    // TODO(fs): In what cases do we need this?
+    bool transformChanged = SVGLayoutSupport::transformToRootChanged(this);
+    SVGLayoutSupport::layoutChildren(this, selfNeedsLayout(), transformChanged);

[pdr., 2016/04/19 18:53:44]

Can you remove this and see what tests fail, and file a bug if it's a real
issue?

[fs, 2016/04/19 19:39:56]

Yepp, I intend to.

     updateCachedBoundaries();
     clearNeedsLayout();
 }
 
 void LayoutSVGHiddenContainer::paint(const PaintInfo&, const LayoutPoint&) const
 {
     // This subtree does not paint.
 }
 
 void LayoutSVGHiddenContainer::absoluteQuads(Vector<FloatQuad>&) const
 {
     // This subtree does not take up space or paint
 }
 
 bool LayoutSVGHiddenContainer::nodeAtFloatPoint(HitTestResult&, const FloatPoint&, HitTestAction)
 {
     return false;
 }
 
 } // namespace blink