2008-08-13 Beth Dakin <bdakin@apple.com>
authorbdakin@apple.com <bdakin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Aug 2008 21:36:15 +0000 (21:36 +0000)
committerbdakin@apple.com <bdakin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Aug 2008 21:36:15 +0000 (21:36 +0000)
commit988ba71356cc3adc7ee209762992e7de19a7ba13
treec57a90f71ec345b9732de5b67bf01030212606d0
parent3737693bb0116317733b36a1c76978c7c1df51d8
2008-08-13  Beth Dakin  <bdakin@apple.com>

        Reviewed by Sam Weinig.

        Fix for <rdar://problem/6141345>

        This patch refines findString and markAllMatchesForText functions'
        interactions with disconnected frames. They no longer rely on
        knowing where a range is relative to the visible region and work
        with disconnected frames that contain frames.

        * editing/Editor.cpp:
        (WebCore::Editor::insideVisibleArea): Now returns a bool instead of
        the visiblity enum.
        (WebCore::Editor::firstVisibleRange): This now returns the very
        first visible range in the document. It's no longer dependent on
        searching forward.
        (WebCore::Editor::lastVisibleRange): This now returns the very last
        visible range in the document. It's no longer dependent on
        searching backwards.
        (WebCore::Editor::nextVisibleRange): This returns the next visible
        range in the appropriate direction from the current range.
        * editing/Editor.h:
        * page/Frame.cpp:
        (WebCore::Frame::findString):
        (WebCore::Frame::markAllMatchesForText):

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