[Webkit-unassigned] [Bug 65898] DeleteWordForward deletes 2 words

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon May 28 18:34:21 PDT 2012


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


Antaryami Pandia (apandia) <antaryami.pandia at motorola.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |antaryami.pandia at motorola.c
                   |                            |om




--- Comment #9 from Antaryami Pandia (apandia) <antaryami.pandia at motorola.com>  2012-05-28 18:34:20 PST ---
I have checked the issue with some other words and below is the observation:

1: foo bar -baz_ --> control+backspace --> foo bar -
2: foo bar @baz_ --> control+backspace --> foo bar @
3: foo bar _baz_ --> control+backspace --> foo

The below code in method "findNextWordFromIndex" is used to calculate the position of character after ctrl+bkspace:-
position = textBreakPreceding(it, position);

case 1 and 2
============
returns the position text just after the special character ie (b) whose position is 9. So the isAlphanumeric check passes and returns the same position.

case 3
======
The position returned is 8, i.e position of (_) . So the isAlphanumeric check fails and it continues to find the next position. And it returns the position of 4 which is the start of 2nd word. So it deletes upto that point.

The "findNextWordFromIndex" method uses the ICU library to determine the position. So I think the ICU library should also returns the correct position when underscore(_) is used.

Also what should be the right output:
foo bar _baz_ --> control+backspace --> foo bar 
OR
foo bar _baz_ --> control+backspace --> foo bar _

Please provide your feedback.

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



More information about the webkit-unassigned mailing list