Currently only RenderBoxes have visual overflow computed per region. This means inline elements are incorrectly handled.
Created attachment 220613 [details] Patch
Comment on attachment 220613 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=220613&action=review > Source/WebCore/rendering/RenderFlowThread.cpp:845 > + return object->isBox() || object->isRenderInline(); There's a FIXME in RenderFlowThread::regionForCompositedLayer: "remove this when we'll have region ranges for inlines as well" > Source/WebCore/rendering/RenderRegion.cpp:718 > + return LayoutRect(); // FIXME: Should ASSERT_NOT_REACHED here? Why the FIXME comment? Just add the assert.
(In reply to comment #2) > (From update of attachment 220613 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=220613&action=review > > > Source/WebCore/rendering/RenderFlowThread.cpp:845 > > + return object->isBox() || object->isRenderInline(); > > There's a FIXME in RenderFlowThread::regionForCompositedLayer: "remove this when we'll have region ranges for inlines as well" This patch doesn't provide ranges for inline elements (e.g. a div with display: inline-block), but the means to access the visual overflow in regions for more elements (RenderInlines in this case). > > > Source/WebCore/rendering/RenderRegion.cpp:718 > > + return LayoutRect(); // FIXME: Should ASSERT_NOT_REACHED here? > > Why the FIXME comment? Just add the assert. Will do.
Created attachment 220637 [details] Patch
Comment on attachment 220637 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=220637&action=review > LayoutTests/fast/regions/overflow-in-variable-width-regions-inline.html:39 > +<p>The paragraph inside the regions must be fragmented. Each fragment must be relatively positioned based on the fragment position.</p> all tests should mention the number of the bug they test.
Created attachment 220702 [details] Patch
Comment on attachment 220702 [details] Patch r=me
Comment on attachment 220702 [details] Patch Clearing flags on attachment: 220702 Committed r161626: <http://trac.webkit.org/changeset/161626>
All reviewed patches have been landed. Closing bug.