Drop transform-change propagation from LayoutSVGHiddenContainer::layout

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

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGHiddenContainer.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGHiddenContainer.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGHiddenContainer.cpp
index 2ef125d58c5cc581fbaae1c02a6fe7c4cfd4a2d5..e613b59093ac9c998ae28a1a1192d2915fc23e53 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGHiddenContainer.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGHiddenContainer.cpp
@@ -33,14 +33,13 @@ 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();