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
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
make mac-wk2-ews happy (11.40 KB, patch)
2014-03-03 00:58 PST, Grzegorz Czajkowski
no flags
Remove two unsupported spelling tests (8.20 KB, patch)
2014-03-05 01:59 PST, Grzegorz Czajkowski
rniwa: review+
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
Note You need to log in before you can comment on or make changes to this bug.