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

Source/core/dom/NodeRenderingContext.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2011 Google 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
@@ skipping 169 matching lines @@
     if (!parentRenderer->canHaveChildren())
         return false;
     if (!m_renderingParent->childShouldCreateRenderer(*this))
         return false;
     return true;
 }
 
 void NodeRenderingContext::moveToFlowThreadIfNeeded()
 {
     ASSERT(m_node->isElementNode());
-    ASSERT(m_style);
+    Element* element = toElement(m_node);

[esprehn, 2013/06/28 20:29:36]

You don't need the ASSERT( before this since toElement does it for you.

+
     if (!RuntimeEnabledFeatures::cssRegionsEnabled())
         return;
 
-    if (m_style->flowThread().isEmpty())
+    if (!m_style)
+        m_style = element->styleForRenderer();
+
+    if (!element->shouldMoveToFlowThread(m_style.get()))
         return;
 
-    // As per http://dev.w3.org/csswg/css3-regions/#flow-into, pseudo-elements such as ::first-line, ::first-letter, ::before or ::after
-    // cannot be directly collected into a named flow.
-    if (m_node->isPseudoElement())
-        return;
+    moveToFlowThread();
+}
 
-    // FIXME: Do not collect elements if they are in shadow tree.
-    if (m_node->isInShadowTree())
-        return;
+void NodeRenderingContext::moveToFlowThread()

[esprehn, 2013/06/28 20:29:36]

Why is this a separate method now? Lets just merge it into
mergeToFlowThreadIfNeeded()

+{
+    ASSERT(m_node->isElementNode());
+    Element* element = toElement(m_node);
 
-    if (m_node->isElementNode() && FullscreenController::isActiveFullScreenElement(toElement(m_node)))
-        return;
-
-    // Allow only svg root elements to be directly collected by a render flow thread.
-    if (m_node->isSVGElement()
-        && (!(m_node->hasTagName(SVGNames::svgTag) && m_node->parentNode() && !m_node->parentNode()->isSVGElement())))
-        return;
-
-    m_flowThread = m_style->flowThread();
+    if (!m_style)
+        m_style = element->styleForRenderer();
+    ASSERT(m_style);
     ASSERT(m_node->document()->renderView());
+    ASSERT(element->shouldMoveToFlowThread(m_style.get()));
     FlowThreadController* flowThreadController = m_node->document()->renderView()->flowThreadController();
-    m_parentFlowRenderer = flowThreadController->ensureRenderFlowThreadWithName(m_flowThread);
+    m_parentFlowRenderer = flowThreadController->ensureRenderFlowThreadWithName(m_style->flowThread());
     flowThreadController->registerNamedFlowContentNode(m_node, m_parentFlowRenderer);
 }
 
 bool NodeRenderingContext::isOnEncapsulationBoundary() const
 {
     return isOnUpperEncapsulationBoundary() || isLowerEncapsulationBoundary(m_parentDetails.insertionPoint()) || isLowerEncapsulationBoundary(m_node->parentNode());
 }
 
 bool NodeRenderingContext::isOnUpperEncapsulationBoundary() const
 {
     return m_node->parentNode() && m_node->parentNode()->isShadowRoot();
 }
 
 void NodeRenderingContext::createRendererForElementIfNeeded()
 {
     ASSERT(!m_node->renderer());
 
     Element* element = toElement(m_node);
 
-    if (!shouldCreateRenderer())
-        return;
+    if (!shouldCreateRenderer()) {
+        // Check the specific case of elements that are children of regions but are flowed into a flow thread themselves.

[esprehn, 2013/06/28 20:29:36]

This whole check should be in a method.

+        bool shouldBeInNamedFlow = false;
+        RenderObject* parentRenderer = this->parentRenderer();
+        if ((parentRenderer && !parentRenderer->canHaveChildren() && parentRenderer->isRenderRegion())
+            || (!parentRenderer && element->parentElement() && element->parentElement()->isDOMChildOfRegion())) {
+            if (!m_style)
+                m_style = element->styleForRenderer();

[esprehn, 2013/06/28 20:29:36]

This doesn't seem right, We should put  if (!m_style) m_style =
element->styleForRenderer(); in one single place, not in lots of methods.

+
+            shouldBeInNamedFlow = element && element->shouldMoveToFlowThread(m_style.get());

[esprehn, 2013/06/28 20:29:36]

element can't be null, you don't need element &&

+
+            // Children of this element will only be allowed to be flowed into other flow-threads if display is NOT none.
+            if (element->rendererIsNeeded(*this))
+                element->setIsDOMChildOfRegion(true);

[esprehn, 2013/06/28 20:29:36]

You never reset this to false so if I remove a child and put it somewhere else
in the tree it's still marked inside regions.

+        }
+
+        if (!shouldBeInNamedFlow)
+            return;
+    }
+
     if (!m_style)
         m_style = element->styleForRenderer();
     ASSERT(m_style);
 
     moveToFlowThreadIfNeeded();
 
     if (!element->rendererIsNeeded(*this))
         return;
 
     RenderObject* parentRenderer = this->parentRenderer();
@@ skipping 59 matching lines @@
     newRenderer->setFlowThreadState(parentRenderer->flowThreadState());
 
     RenderObject* nextRenderer = this->nextRenderer();
     textNode->setRenderer(newRenderer);
     // Parent takes care of the animations, no need to call setAnimatableStyle.
     newRenderer->setStyle(m_style.release());
     parentRenderer->addChild(newRenderer, nextRenderer);
 }
 
 }