AX: WebKit is returning the wrong rangeForLine

[Nan Wang]

In content editable with multiple lines, webkit is giving a wrong range for line and omitting the first character of that line.

<rdar://problem/30942088>

[Nan Wang]

Created attachment 305061 [details]
patch

[chris fleizach]

Comment on attachment 305061 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=305061&action=review

> LayoutTests/accessibility/mac/range-for-contenteditable-newline.html:54
> +    var rangeForLine = textControl3.rangeForLine(1);

do we need to worry about lineForRange as well? we should probably add some test just to be sure

[chris fleizach]

Comment on attachment 305061 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=305061&action=review

> LayoutTests/accessibility/mac/range-for-contenteditable-newline.html:17
> +<p>jumped over the lazy dog</p>

if we put a <pre> inside of a content editable and then just used regular newlines, do we have any other issues

[Nan Wang]

Created attachment 305114 [details]
patch

updated test

[WebKit Commit Bot]

Comment on attachment 305114 [details]
patch

Clearing flags on attachment: 305114

Committed r214274: <http://trac.webkit.org/changeset/214274>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.