[Webkit-unassigned] [Bug 55650] New: VisiblePosition maybe at an upstream position even if its affinity is downstream

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=55650

           Summary: VisiblePosition maybe at an upstream position even if
                    its affinity is downstream
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Enhancement
          Priority: P2
         Component: HTML Editing
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: eric at webkit.org, justin.garcia at apple.com,
                    xji at chromium.org, enrica at apple.com,
                    kalman at chromium.org, leviw at chromium.org
            Blocks: 52098


While I was working on the bug 55098, I realized that VisiblePosition can be upstream even if its affinity claims to be downstream.  This bug is due to the following lines in http://trac.webkit.org/browser/trunk/Source/WebCore/editing/VisiblePosition.cpp.

456        Position candidate = position.upstream();
457        if (candidate.isCandidate())
458            return candidate;
459        candidate = position.downstream();
460        if (candidate.isCandidate())
461            return candidate;

It returns an upstream position but it doesn't modify its affinity so we end up with a VisiblePosition that stores an upstream position but claims to be a downstream position.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.