RESOLVED FIXED 129482
Remove unsupported spelling tests. 
https://bugs.webkit.org/show_bug.cgi?id=129482
Summary Remove unsupported spelling tests.
Grzegorz Czajkowski
Reported 2014-02-28 06:44:24 PST
Skip two spelling tests that require spellchecking of multiple words. Currently, neither WebKit port is supporting this feature. Those tests were added by Chromium at r141354.
Attachments
Patch (11.00 KB, patch)
2014-02-28 06:52 PST, Grzegorz Czajkowski 		no flags
DetailsFormatted DiffDiff
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2 (458.91 KB, application/zip)
2014-02-28 07:51 PST, Build Bot 		no flags
Details
make mac-wk2-ews happy (11.40 KB, patch)
2014-03-03 00:58 PST, Grzegorz Czajkowski 		no flags
DetailsFormatted DiffDiff
Remove two unsupported spelling tests (8.20 KB, patch)
2014-03-05 01:59 PST, Grzegorz Czajkowski 		rniwa: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Grzegorz Czajkowski
Comment 1 2014-02-28 06:52:18 PST
Created attachment 225459 [details] Patch I requested review since nonsignificant changes in those tests
Build Bot
Comment 2 2014-02-28 07:51:53 PST
Comment on attachment 225459 [details] Patch Attachment 225459 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/6495454632083456 New failing tests: editing/spelling/spelling-exactly-selected-word.html
Build Bot
Comment 3 2014-02-28 07:51:54 PST
Created attachment 225462 [details] Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-09 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Grzegorz Czajkowski
Comment 4 2014-03-03 00:58:53 PST
Created attachment 225636 [details] make mac-wk2-ews happy
Ryosuke Niwa
Comment 5 2014-03-05 01:00:53 PST
Can we just remove these tests?
Grzegorz Czajkowski
Comment 6 2014-03-05 01:59:40 PST
Created attachment 225872 [details] Remove two unsupported spelling tests
Grzegorz Czajkowski
Comment 7 2014-03-05 02:30:10 PST
Committed r165102: <http://trac.webkit.org/changeset/165102>
Note You need to log in before you can comment on or make changes to this bug.