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

Source/core/dom/PseudoElement.h

Index: Source/core/dom/PseudoElement.h
diff --git a/Source/core/dom/PseudoElement.h b/Source/core/dom/PseudoElement.h
index 87c80647c180179a24ab161e03a90b9461b3dc61..013973239527e7679fe7470acf4c364b3bc48494 100644
--- a/Source/core/dom/PseudoElement.h
+++ b/Source/core/dom/PseudoElement.h
@@ -46,6 +46,14 @@ public:
     virtual void attach() OVERRIDE;
     virtual bool rendererIsNeeded(const NodeRenderingContext&) OVERRIDE;
 
+    // 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.
+    virtual bool moveToFlowThreadIsNeeded(RefPtr<RenderStyle>& cachedStyle) OVERRIDE

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

I am really not a huge fan of this function naming. Usually such functions
starts with "should" and here it would make IMO a better name:
shouldMoveElementToFlowThread

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

Ok, I will rename it.
PS: it may be possible in a future spec version to also flow nodes, not just
elements. So I would drop the "Element" word from the suggested name.

+    {
+        UNUSED_PARAM(cachedStyle);

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

If you didn't put the argument in the function signature, you wouldn't need this
line.

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

That's true when declaring the function. When the implementation is written in
the header at the declaration point, a missing argument would make the compiler
think it's a different method signature.

[Julien - ping for review, 2013/05/10 00:13:05]

I don't think you are right (argument names don't impact the signature, only the
number of arguments and their types). See among other example RenderBlock.h:

virtual void adjustInlineDirectionLineBounds(int /* expansionOpportunityCount
*/, float& /* logicalLeft */, float& /* logicalWidth */) const { }

+        return false;
+    }
+
     virtual bool canStartSelection() const OVERRIDE { return false; }
     virtual bool canContainRangeEndPoint() const OVERRIDE { return false; }