Bug 49873 – [chromium] editing/selection/extend-selection-home-end.html should have one expected result

• Status: RESOLVED
• Resolution: FIXED
• Component: HTML Editing

[Ryosuke Niwa]

editing/selection/extend-selection-home-end.html has a different expected result on chromium because the last two test specify the width by pixel:

http://trac.webkit.org/browser/trunk/LayoutTests/editing/selection/resources/extend-selection.js
['home-end', 'line-break:before-white-space; width:100px', 'abcdefg abcdefg abcdefg a abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg '],
['home-end', 'line-break:after-white-space; width:100px', 'abcdefg abcdefg abcdefg a abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg abcdefg ']

We should replace these values by em so that chromium gets the same expected result as other platforms.

[Benjamin (Ben) Kalman]

This is complicated and possibly blocked (?) by https://bugs.webkit.org/show_bug.cgi?id=51163

[Benjamin (Ben) Kalman]

(In reply to comment #1)
> This is complicated and possibly blocked (?) by https://bugs.webkit.org/show_bug.cgi?id=51163

I should expand on what I mean here.

There should only be 2 different results needed for this test, win/linux and mac, since the mac has different home/end behaviour.  So it should just work to have a default expected.txt, and a specialised one in platform/mac.

Currently there are 5.  Some of these can be reduced by changing 100px to e.g. 5em (as Ryosuke says), but not all, given the test (dumprendertree?  not sure) bug with running gtk tests.

[Benjamin (Ben) Kalman]

Created attachment 76970 [details]
Patch

[Benjamin (Ben) Kalman]

Unfortunately there are actually 3 results needed.
 - default results
 - mac results (since line boundary selection behaviour is different on a mac apparently)
 - win results that are the same as the default results, because otherwise chromium-{linux,win} falls back to mac

[Antonio Gomes]

Comment on attachment 76970 [details]
Patch

LGTM.

I ran the patch locally on Qt and Gtk and both succeeded the test.

[WebKit Commit Bot]

Comment on attachment 76970 [details]
Patch

Clearing flags on attachment: 76970

Committed r74331: <http://trac.webkit.org/changeset/74331>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 76970 [details]:

inspector/debugger-pause-on-breakpoint.html bug 51320 (author: [e-mail])
The commit-queue is continuing to process your patch.