Summary: | [Efl] Enable spellchecking tests added in r141354 | ||
---|---|---|---|
Product: | WebKit | Reporter: | Rouslan Solomakhin <rouslan+webkit> |
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED WONTFIX | ||
Severity: | Normal | CC: | g.czajkowski, mcatanzaro, rniwa, rouslan+webkit, tony |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | 112554, 81042, 107682, 107840, 108172, 108405, 109404, 112426, 113742 | ||
Bug Blocks: |
Description
Rouslan Solomakhin
2013-01-31 14:33:35 PST
Hi, Unfortunately, these tests can not be enabled for EFL (WK2) as our implementation lack of features required by the tests: - smartInsertDeleteEnabled, - unified text checker, - asynchronous spell checking. BTW, do those tests really need asynchronous spell checking? It seems they in vast majority test selection/white space matters (please correct me if I am wrong). One of the added test requires selection to get suggestions for the misspelled word. Linux WebKit ports omit this policy (bug 103520). Perhaps we can do "if (testRunner.smartInsertDeleteEnabled) testRunner.smartInsertDeleteEnabled(true);" ? (In reply to comment #1) > BTW, do those tests really need asynchronous spell checking? It seems they in vast majority test selection/white space matters (please correct me if I am wrong). My original goal with the tests was to make Chromium port work better. Chromium uses async spellcheck on all operating systems except Mac. Perhaps we can split the tests into two groups: async and sync? (In reply to comment #3) > (In reply to comment #1) > > BTW, do those tests really need asynchronous spell checking? It seems they in vast majority test selection/white space matters (please correct me if I am wrong). > > My original goal with the tests was to make Chromium port work better. Chromium uses async spellcheck on all operating systems except Mac. Perhaps we can split the tests into two groups: async and sync? It'd great! As far as I know EFL and GTK+ haven't implemented asynchronous spell checking yet so it prevents to run spelling and editing aspects provided by your tests. (In reply to comment #2) > Perhaps we can do "if (testRunner.smartInsertDeleteEnabled) testRunner.smartInsertDeleteEnabled(true);" ? IMHO that would be more convincing and it's likely to pass the tests which do not require 'smartInsertDelete' feature. Howdy folks. I plan to move these tests into editing/spelling/unified-async, so that platforms can enable these tests when/if they implement unified text checking and async spell checking. Is it alright if I close this bug as WontFix, because it's redundant to your bug https://bugs.webkit.org/show_bug.cgi?id=108172 ? (In reply to comment #5) > Howdy folks. I plan to move these tests into editing/spelling/unified-async, so that platforms can enable these tests when/if they implement unified text checking and async spell checking. Is it alright if I close this bug as WontFix, because it's redundant to your bug https://bugs.webkit.org/show_bug.cgi?id=108172 ? Hi, Please don't close this bug. It traces enabling these spell checking tests for EFL. I hope to unskip than as soon as dependency land. Sounds good. spelling-exactly-selected-word.html and spelling-with-punctuation-selection.html unskipped in r149730. spelling-with-whitespace-selection.html unskipped in r148421. spelling-double-clicked-word.html unskipped in r150363. Closing this bug because the EFL port has been removed from trunk. If you feel this bug applies to a different upstream WebKit port and was closed in error, please either update the title and reopen the bug, or leave a comment to request this. |