Reviewed by Hyatt
authorkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Oct 2004 18:52:39 +0000 (18:52 +0000)
committerkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Oct 2004 18:52:39 +0000 (18:52 +0000)
commit0d7f81766d4ae9fb3eb731aab686ae4fcd1f04af
tree0c1dcf6ba22c7bf432861d681b79af61e528e06d
parent5a6c9a0edad9358d348777fb5479b7887d331ad9
    Reviewed by Hyatt

        Fix for this bug:

        <rdar://problem/3825289> REGRESSION (Mail): Crash in fontForSelection in empty window

        * khtml/editing/visible_position.cpp:
        (khtml::VisiblePosition::previousPosition): Switch to node iteration instead "leaf"
        iteration. I have been wanting to make this change for a long time, but couldn't
        since other code relied on the leaf behavior. That is no longer true. Plus, the
        bug fix requires the new behavior.
        (khtml::VisiblePosition::nextPosition): Ditto.
        (khtml::VisiblePosition::isCandidate): Empty blocks needed a height to be a candidate,
        but we make a special case for the body element. This fixes the bug.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@7753 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog-2005-08-23
WebCore/khtml/editing/visible_position.cpp