(CANCELED) Fix TextAutosizer not to scheduleRelayout()

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

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGText.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGText.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGText.cpp
index 9413c6868a932a243361ee2fae82e3477ece1982..21637fa5ca261bdd7228adbfcdf9003fbc11fa92 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGText.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGText.cpp
@@ -238,7 +238,7 @@ void LayoutSVGText::layout()
     // Reduced version of LayoutBlock::layoutBlock(), which only takes care of SVG text.
     // All if branches that could cause early exit in LayoutBlocks layoutBlock() method are turned into assertions.
     ASSERT(!isInline());
-    ASSERT(!simplifiedLayout());
+    ASSERT(!simplifiedLayout(nullptr));

[cbiesinger, 2016/04/15 17:08:08]

I don't really know much about SVG but this seems weird?

[kojii, 2016/04/15 17:58:44]

By "weird" do you mean the original code is weird or the change is weird?

I don't know much either, but LayoutSVGText does its layout by its own, using
platform/fonts and shaper directly, without using most of functions in
LayoutText, LineBreaker, etc.

As far as I debugged a few cases, all following 4 bits are false, so
simplifiedLayout() returns false immediately.
normalChildNeedsLayout()
posChildNeedsLayout()
needsSimplifiedNormalFlowLayout()
needsPositionedMovementLayout()

So if it's asserting that, use null SubtreeLayoutScope looks fine to me.

Can you elaborate a bit more?

     ASSERT(!scrollsOverflow());
     ASSERT(!hasControlClip());
     ASSERT(!positionedObjects());