• commit-queue@webkit.org's avatar
    ASSERT failure due to combine-text with preceding spaces · f6c35c0b
    commit-queue@webkit.org authored
    Patch by Ken Buchanan <kenrb@chromium.org> on 2012-01-06
    Reviewed by David Hyatt.
    A couple of ASSERTs were failing due to a parsing problem when
    advancing an inline iterator to the next linebreak in a
    RenderCombineText. skipLeadingWhitespace advances the iterator
    over leading whitespace but when searching for the line break
    nextLineBreak would call RenderCombineText::combineText(),
    collapsing the text so that the iterator is pointing past the
    end of it.
    This patch causes combineText() to be called during
    skipLeadingWhiteSpace before iteration over the RenderCombineText
    * rendering/RenderBlockLineLayout.cpp:
    New test with combine-text that begins with mutiple spaces,
    showing ASSERT failures in bug 65147.
    * fast/text/international/spaces-combined-in-vertical-text.html: Added
    * fast/text/international/spaces-combined-in-vertical-text-expected.txt: Added
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@104322 268f45cc-cd09-0410-ab3c-d52691b4dbfc
spaces-combined-in-vertical-text.html 352 Bytes