Simulate behavior of Learn and Ignore Word in the text field.
Created attachment 206084 [details] proposed patch
Comment on attachment 206084 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=206084&action=review > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_text_checker.cpp:807 > + knownWord = ""; knownWord = String()? > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_text_checker.cpp:839 > + knownWord = ""; knownWord = String()?
Comment on attachment 206084 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=206084&action=review > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_text_checker.cpp:55 > +String knownWord; why not static? >> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_text_checker.cpp:807 >> + knownWord = ""; > > knownWord = String()? emptyString() ?
Created attachment 206088 [details] apply suggestions
> >> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_text_checker.cpp:807 > >> + knownWord = ""; > > > > knownWord = String()? > > emptyString() ? Right, emptyString() is better indeed.
Why no review flag?
Created attachment 206090 [details] apply suggestions
Comment on attachment 206090 [details] apply suggestions Ok, r=me but please let Grzegorz have a look before landing.
Comment on attachment 206090 [details] apply suggestions View in context: https://bugs.webkit.org/attachment.cgi?id=206090&action=review I'd love to see it in the trunk. Added one comments regarding to the spellchecking and one nit. > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_text_checker.cpp:188 > + *misspellingLength = static_cast<int32_t>(knownWord.length()); As I understand correctly, this callback should treat known words as spelled correctly. Therefore, its length should be set to 0 to notify WebCore to don't create spelling marker for it. Additionally, the location should be set to -1 according to the doc. > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_text_checker.cpp:813 > + mouseDoubleClick(10, 20, 1 /* Left button */); Can we skip left button value and this comment here? mouseDoubleClick passes left button implicitly. > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_text_checker.cpp:845 > + mouseDoubleClick(10, 20, 1 /* Left button */); Ditto.
Created attachment 206127 [details] apply suggestions
Comment on attachment 206127 [details] apply suggestions Thanks
Comment on attachment 206127 [details] apply suggestions Clearing flags on attachment: 206127 Committed r152414: <http://trac.webkit.org/changeset/152414>
All reviewed patches have been landed. Closing bug.