Reviewed by Darin Adler.
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Apr 2006 17:51:54 +0000 (17:51 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Apr 2006 17:51:54 +0000 (17:51 +0000)
commit95bb7fff1b1c9c555b7c0b40fcc174cc6d0cd404
tree41d70fa5d220c3ba22a2caed073a89786192e1af
parent5b3f8ace41e600feb679b616e607f7fba46f8d3b
    Reviewed by Darin Adler.

        - fixed <rdar://problem/4509328> highlight all matches hangs when searching for tab character on www.google.com (and others)

        For reasons not yet completely understood, searching for a tab character on some pages (Google, Amazon) finds a match
        with a non-collapsed range but then claims that the end visible position of the match is the original start visible
        position of the search range. This was causing the highlightAllMatches code to loop forever. Fixed the loop by
        checking for the non-advancing search range explicitly. I'm going to track down a reduction of the bogus
        search-for-tab issue, and write that up as a separate bug (that bug is not a regression; you can "find" a tab on
        google in Tiger also).

        * page/Frame.cpp:
        (WebCore::Frame::highlightAllMatchesForString):
        break the loop if the search range hasn't advanced

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13822 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog
WebCore/page/Frame.cpp