Bug 129482

Summary: Remove unsupported spelling tests.
Product: WebKit Reporter: Grzegorz Czajkowski <g.czajkowski>
Component: HTML EditingAssignee: Grzegorz Czajkowski <g.czajkowski>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2
none
make mac-wk2-ews happy
none
Remove two unsupported spelling tests rniwa: review+

Description Grzegorz Czajkowski 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.
Comment 1 Grzegorz Czajkowski 2014-02-28 06:52:18 PST
Created attachment 225459 [details]
Patch

I requested review since nonsignificant changes in those tests
Comment 2 Build Bot 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
Comment 3 Build Bot 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
Comment 4 Grzegorz Czajkowski 2014-03-03 00:58:53 PST
Created attachment 225636 [details]
make mac-wk2-ews happy
Comment 5 Ryosuke Niwa 2014-03-05 01:00:53 PST
Can we just remove these tests?
Comment 6 Grzegorz Czajkowski 2014-03-05 01:59:40 PST
Created attachment 225872 [details]
Remove two unsupported spelling tests
Comment 7 Grzegorz Czajkowski 2014-03-05 02:30:10 PST
Committed r165102: <http://trac.webkit.org/changeset/165102>