Drop transform-change propagation from LayoutSVGHiddenContainer::layout

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);
-    // TODO(fs): In what cases do we need this?
-    bool transformChanged = SVGLayoutSupport::transformToRootChanged(this);
 
     // When hasRelativeLengths() is false, no descendants have relative lengths
     // (hence no one is interested in viewport size changes).
     bool layoutSizeChanged = element()->hasRelativeLengths()
         && SVGLayoutSupport::layoutSizeOfNearestViewportChanged(this);
 
+    const bool transformChanged = false;

[Stephen Chennney, 2016/04/20 16:09:10]

I think it's OK to just pass false to the method, rather than create the temp.

[fs, 2016/04/20 16:42:48]

Since this is the "odd one out" wrt this I guess it should be fine. Done.

     SVGLayoutSupport::layoutChildren(firstChild(), selfNeedsLayout(), transformChanged, layoutSizeChanged);
     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