Fix assertion: !visualMetricsValues.isEmpty() in layoutTextOnLineOrPath

Source/core/dom/Text.cpp

Index: Source/core/dom/Text.cpp
diff --git a/Source/core/dom/Text.cpp b/Source/core/dom/Text.cpp
index 1fb61a41933ca5b128f5ec92cd34aad7946dc815..b181f59756e418769f105ecca2492158f614f1ce 100644
--- a/Source/core/dom/Text.cpp
+++ b/Source/core/dom/Text.cpp
@@ -257,6 +257,9 @@ bool Text::textRendererIsNeeded(const RenderStyle& style, const RenderObject& pa
     if (!parent.canHaveWhitespaceChildren())
         return false;
 
+    if (style.whiteSpace() == PRE_WRAP && parent.isSVG()) // pre-wrap in SVG never makes renderer.

[kouhei (in TOK), 2014/12/09 09:15:23]

Should we also care about pre/pre-line?

[fs, 2014/12/09 10:18:32]

All cases in which the metrics builder will end up collapsing I suppose - ie.
everything not 'pre'. (SVG2 will change this though.)

+        return false;
+
     if (style.preserveNewline()) // pre/pre-wrap/pre-line always make renderers.
         return true;