[CSS Regions] Elements in a region should be assignable to a named flow

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 184 matching lines @@
 Node::NodeType Text::nodeType() const
 {
     return TEXT_NODE;
 }
 
 PassRefPtr<Node> Text::cloneNode(bool /*deep*/)
 {
     return cloneWithData(data());
 }
 
-bool Text::textRendererIsNeeded(const NodeRenderingContext& context)

[Julien - ping for review, 2013/05/09 15:30:18]

Again, losing const-ness makes me sad.

[Mihai Maerean, 2013/05/09 16:25:55]

Me too, but there's no other way to make the compiler happy.

+bool Text::textRendererIsNeeded(NodeRenderingContext& context)
 {
     if (isEditingText())
         return true;
 
     if (!length())
         return false;
 
     if (context.style()->display() == NONE)
         return false;
 
@@ skipping 84 matching lines @@
             reattach();
     }
     clearNeedsStyleRecalc();
 }
 
 void Text::updateTextRenderer(unsigned offsetOfReplacedData, unsigned lengthOfReplacedData)
 {
     if (!attached())
         return;
     RenderText* textRenderer = toRenderText(renderer());
-    if (!textRenderer || !textRendererIsNeeded(NodeRenderingContext(this, textRenderer->style()))) {
+    if (!textRenderer) {
         reattach();
         return;
     }
+    NodeRenderingContext renderingContext(this, textRenderer->style());
+    if (!textRendererIsNeeded(renderingContext)) {
+        reattach();
+        return;
+    }
     textRenderer->setTextWithOffset(dataImpl(), offsetOfReplacedData, lengthOfReplacedData);
 }
 
 bool Text::childTypeAllowed(NodeType) const
 {
     return false;
 }
 
 PassRefPtr<Text> Text::cloneWithData(const String& data)
 {
@@ skipping 27 matching lines @@
             result.appendLiteral("; ");
         result.appendLiteral("value=");
         result.append(s);
     }
 
     strncpy(buffer, result.toString().utf8().data(), length - 1);
 }
 #endif
 
 } // namespace WebCore