Bug 108528

Summary: [Efl] Enable spellchecking tests added in r141354
Product: WebKit Reporter: Rouslan Solomakhin <rouslan+webkit>
Component: Tools / TestsAssignee: 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
http://trac.webkit.org/changeset/141354 added a whole bunch of spellchecking tests. We should enable these on Efl.
Comment 1 Grzegorz Czajkowski 2013-02-01 00:53:09 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).
Comment 2 Rouslan Solomakhin 2013-02-01 09:40:07 PST
Perhaps we can do "if (testRunner.smartInsertDeleteEnabled) testRunner.smartInsertDeleteEnabled(true);" ?
Comment 3 Rouslan Solomakhin 2013-02-01 09:41:54 PST
(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?
Comment 4 Grzegorz Czajkowski 2013-02-04 23:55:53 PST
(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.
Comment 5 Rouslan Solomakhin 2013-02-22 17:32:32 PST
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 ?
Comment 6 Grzegorz Czajkowski 2013-03-01 07:46:32 PST
(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.
Comment 7 Rouslan Solomakhin 2013-03-01 09:47:25 PST
Sounds good.
Comment 8 Grzegorz Czajkowski 2013-05-17 00:20:11 PDT
spelling-exactly-selected-word.html and spelling-with-punctuation-selection.html unskipped in r149730.
    
spelling-with-whitespace-selection.html unskipped in r148421.
Comment 9 Grzegorz Czajkowski 2013-05-20 03:21:11 PDT
spelling-double-clicked-word.html unskipped in r150363.
Comment 10 Michael Catanzaro 2017-03-11 10:38:03 PST
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.