Reviewed by Harrison
authorkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 Nov 2004 16:58:30 +0000 (16:58 +0000)
committerkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 Nov 2004 16:58:30 +0000 (16:58 +0000)
commit771c75b9b01a76be1f653a81d36a62e390e00a77
tree1fd5df98514552a106113c6ced74a755e8a733ef
parentc68b2edcc6a90c165e399e6a7eb3f081b5a89e98
    Reviewed by Harrison

        Some cleanups and fixes in upstream and downstream functions.

        Removed redundant checks for isBlockFlow() when calling enclosingBlockFlowElement().
        Blocks do not need to skip the call to enclosingBlockFlowElement() for fear that the
        block's enclosing block will be returned.

        Remove code from upstream that confined the serach to block boundaries outside of
        the code which runs in the StayInBlock case. This code was redundant, and caused
        incorrect results to be returned in the DoNotStayInBlock case.

        In downstream, the check for crossing into a new block should use the equivalentDeepPosition()
        node, not the the this pointer's node.

        * khtml/xml/dom_position.cpp:
        (DOM::Position::upstream)
        (DOM::Position::downstream)

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