Fix assertion: !visualMetricsValues.isEmpty() in layoutTextOnLineOrPath

Source/core/dom/Text.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  *
  * 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.
  *
@@ skipping 239 matching lines @@
 
     if (style.display() == NONE)
         return false;
 
     if (!containsOnlyWhitespace())
         return true;
 
     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;
 
     const RenderObject* prev = NodeRenderingTraversal::previousSiblingRenderer(this);
     if (prev && prev->isBR()) // <span><br/> <br/></span>
         return false;
 
     if (parent.isRenderInline()) {
         // <span><div/> <div/></span>
         if (prev && !prev->isInline())
@@ skipping 131 matching lines @@
             result.appendLiteral("; ");
         result.appendLiteral("value=");
         result.append(s);
     }
 
     strncpy(buffer, result.toString().utf8().data(), length - 1);
 }
 #endif
 
 } // namespace blink