Created attachment 104667 [details] Test file LineWidth::shrinkAvailableWidthForNewFloatIfNeeded is updating the m_left, m_right properties, but is not taking into account that m_isFirstLine is true and the text indent may apply. The break iterator will actually think that the available width is very large, but RenderBlock::computeInlineDirectionPositionsForLine is using the correct values. See test document for more info.
Created attachment 104798 [details] Patch V1
Comment on attachment 104798 [details] Patch V1 View in context: https://bugs.webkit.org/attachment.cgi?id=104798&action=review > LayoutTests/ChangeLog:11 > + * fast/block/float/floats-and-text-indent.html: Added. Please put expectation files under platform directory. Layout related tests are rarely portable between platforms.
Created attachment 104825 [details] Patch V2 Moved expected results to platform mac.
Comment on attachment 104825 [details] Patch V2 r=me
Comment on attachment 104825 [details] Patch V2 Clearing flags on attachment: 104825 Committed r93626: <http://trac.webkit.org/changeset/93626>
All reviewed patches have been landed. Closing bug.