CTTE: RenderNamedFlowThread and FlowThreadController should operate on Elements, not Nodes
Created attachment 212306 [details] Patch
Comment on attachment 212306 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=212306&action=review r=me > Source/WebCore/rendering/RenderNamedFlowThread.cpp:556 > +static bool isContainedInElements(Vector<Element*> others, Element* element) 'others' should be a const reference, we are making a silly copy here.
Committed r156250: <http://trac.webkit.org/changeset/156250>
According to the CSS Regions Specification (http://dev.w3.org/csswg/css-regions/#the-flow-into-property), you can collect nodes (not just elements) into a flow thread. This bug is invalid. The named flow thread below will only contain a text node, the child of the div marked .content. <style> .content { flow-into:flow content; } .region { flow-from:flow; } </style> <div class="content">some text</div> <div class="region"></div>