WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
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
Details
Formatted Diff
Diff
Remove two unsupported spelling tests
(8.20 KB, patch)
2014-03-05 01:59 PST
,
Grzegorz Czajkowski
rniwa
: review+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug