Bug 19443

Summary:

REGRESSION (r32531-r32652): Moving the insertion point past a newline inserted in a text area skips the next character

Product:

WebKit

Reporter:

mitz

Component:

HTML Editing

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

Keywords:

InRadar, Regression

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

Attachments:

Description	Flags
Patch	justin.garcia: review+

Description mitz 2008-06-08 20:47:57 PDT

Steps to reproduce:
1. In a text area, such as on this page, type "foo", press return, and type "bar" on the next line.
2. Place the insertion point on the first line, after "foo".
3. Press the right arrow key.

Expected results:
The insertion point should move to the next line, just before the "b".

Actual results:
The insertion point moved to the next line, between "b" and "ar".

Regression:
bisect-builds puts this in the r32531-r32652 range. The visual arrow keys (for bidi text) patch was landed in that range.

Note:
This does not seem to happen with a content-editable div or with a text area already containing the line break.

Comment 1 mitz 2008-06-08 20:48:33 PDT

<rdar://problem/5994544>

Comment 2 mitz 2008-06-08 20:50:48 PDT

(In reply to comment #0)
> The visual arrow keys (for
> bidi text) patch was landed in that range.

<http://trac.webkit.org/changeset/32605>.

Comment 3 mitz 2008-06-09 21:01:58 PDT

Created attachment 21599 [details]
Patch

Comment 4 Justin Garcia 2008-06-09 21:54:09 PDT

Comment on attachment 21599 [details]
Patch

r=me

Comment 5 mitz 2008-06-09 21:59:54 PDT

Fixed in <http://trac.webkit.org/changeset/34475>.