Include 'outline' in LayoutSVGBlock::visualOverflowRect

Source/core/layout/svg/LayoutSVGText.cpp

Index: Source/core/layout/svg/LayoutSVGText.cpp
diff --git a/Source/core/layout/svg/LayoutSVGText.cpp b/Source/core/layout/svg/LayoutSVGText.cpp
index f23d3ea0eaba584b72dbe7869022526a333e430b..4a03d9c9302811d112693a37c278decbf3213bf0 100644
--- a/Source/core/layout/svg/LayoutSVGText.cpp
+++ b/Source/core/layout/svg/LayoutSVGText.cpp
@@ -369,7 +369,6 @@ void LayoutSVGText::layout()
     ASSERT(!hasControlClip());
     ASSERT(!hasColumns());
     ASSERT(!positionedObjects());
-    ASSERT(!m_overflow);
     ASSERT(!isAnonymousBlock());
 
     if (!firstChild())
@@ -397,6 +396,9 @@ void LayoutSVGText::layout()
     if (!firstLineBox())
         setFrameRect(LayoutRect());
 
+    m_overflow.clear();
+    addVisualEffectOverflow();

[chrishtr, 2015/06/04 16:57:03]

So now LayoutSVGBlock inherits behavior from LayoutBlockFlow and gets its
overflow from there?

[fs, 2015/06/05 09:01:47]

If the derived class does not override that in any way - yes. LayoutSVGText
overrides this (does not call parent-class layout), while LayoutSVGForeignObject
doesn't override (and did update m_overflow before, but didn't use it in
visualOverflowRect - which might seem a bit odd.)

+
     if (!updateCachedBoundariesInParents)
         updateCachedBoundariesInParents = oldBoundaries != objectBoundingBox();