[Webkit-unassigned] [Bug 23232] New: Problems with Selection::appendTrailingWhitespace()

[bugzilla-daemon at webkit.org]

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

           Summary: Problems with Selection::appendTrailingWhitespace()
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Mac OS X 10.5
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: eroman at chromium.org


There are some problems with Selection::appendTrailingWhitespace():

1. May set endpoint to null node, causing crash in WebCore::Selection::toRange.

(This is because VisiblePosition::next() doesn't necessarily advance to same
place as VisiblePosition::characterAfter(), especially if it is a non-visible
character.).

2. Should match NBSP as whitespace.

3. Should search for whitepsace in sibling nodes too.
For example:
  <div><span>doubleclickme</span> </div>
Should grow selection to include the whitespace outside of the span element (to
be consistent with IE/FF).

<background>
Selection::appendTrailingWhitespace() is a codepath used only by Chromium, see
http://trac.webkit.org/changeset/38735 which introduced it.
</background>


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