Bug 108405

Summary: Tests for spellcheck behavior
Product: WebKit Reporter: Rouslan Solomakhin <rouslan+webkit>
Component: HTML EditingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: groby, gyuyoung.kim, rakuco, rniwa, rouslan+webkit, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 108535, 108536, 106815, 108528, 108529, 108530    
Attachments:
Description Flags
Patch none

Rouslan Solomakhin
Reported 2013-01-30 16:47:19 PST
Add tests for the following spellcheck behavior: - Spellcheck selects the misspelled word on context click. - Spelling should work when the user selects the misspelled word exactly. - If the word is not misspelled, there should be no spelling options in the context menu. - Spellcheck should select multi-word misspellings on context click. - Spelling should work when the user selects a multi-word misspelling exactly. - Spelling should work for double-clicked misspellings. - Spelling should ignore whitespace. - Underscores should be treated as whitespace: spelling should ignore them. - Punctuation marks should be treated as whitespace: spelling should ignore them. - Spelling should be disabled when user selects a part of misspelling. - Spelling should be disabled when user selects multiple words that are not a single misspelling.
Attachments
Patch (60.54 KB, patch)
2013-01-30 17:02 PST, Rouslan Solomakhin
no flags
Rouslan Solomakhin
Comment 1 2013-01-30 17:02:27 PST
Rouslan Solomakhin
Comment 2 2013-01-30 17:04:44 PST
Comment on attachment 185627 [details] Patch This is the layout-test part of https://webkit.org/b/106815. I am using [Skip] instead of [Failure] on these tests because some of these tests are already passing.
WebKit Review Bot
Comment 3 2013-01-30 18:09:33 PST
Comment on attachment 185627 [details] Patch Clearing flags on attachment: 185627 Committed r141354: <http://trac.webkit.org/changeset/141354>
WebKit Review Bot
Comment 4 2013-01-30 18:09:38 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.