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.
This is complicated and possibly blocked (?) by https://bugs.webkit.org/show_bug.cgi?id=51163
(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.
Created attachment 76970 [details] Patch
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
Comment on attachment 76970 [details] Patch LGTM. I ran the patch locally on Qt and Gtk and both succeeded the test.
Comment on attachment 76970 [details] Patch Clearing flags on attachment: 76970 Committed r74331: <http://trac.webkit.org/changeset/74331>
All reviewed patches have been landed. Closing bug.
The commit-queue encountered the following flaky tests while processing attachment 76970 [details]: inspector/debugger-pause-on-breakpoint.html bug 51320 (author: podivilov@chromium.org) The commit-queue is continuing to process your patch.