<br> nodes should not be allowed within SVG text

Source/core/rendering/svg/RenderSVGText.cpp

 /*
  * Copyright (C) 2006 Apple Computer, Inc.
  * Copyright (C) 2006 Alexander Kellett <lypanov@kde.org>
  * Copyright (C) 2006 Oliver Hunt <ojh16@student.canterbury.ac.nz>
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2010-2012. All rights reserved.
  * Copyright (C) 2012 Google Inc.
  *
@@ skipping 52 matching lines @@
 {
 }
 
 RenderSVGText::~RenderSVGText()
 {
     ASSERT(m_layoutAttributes.isEmpty());
 }
 
 bool RenderSVGText::isChildAllowed(RenderObject* child, RenderStyle*) const
 {
-    return child->isSVGInline() || (child->isText() && !SVGRenderSupport::isEmptySVGInlineText(child));
+    return (child->isText() && SVGRenderSupport::isRenderableTextNode(child)) || child->isSVGInline();

[Stephen Chennney, 2014/02/03 16:01:15]

Slightly more efficient to order the || clause the other way, right? You don't
expect one or other clause to be true more often, and the isSVGInline is
cheaper.

 }
 
 RenderSVGText* RenderSVGText::locateRenderSVGTextAncestor(RenderObject* start)
 {
     ASSERT(start);
     while (start && !start->isSVGText())
         start = start->parent();
     if (!start || !start->isSVGText())
         return 0;
     return toRenderSVGText(start);
@@ skipping 453 matching lines @@
     return 0;
 }
 
 // Fix for <rdar://problem/8048875>. We should not render :first-letter CSS Style
 // in a SVG text element context.
 void RenderSVGText::updateFirstLetter()
 {
 }
 
 }