[webkit-reviews] review granted: [Bug 109577] [WK2] Asynchronous spell checking implementation : [Attachment 190027] apply Anders' suggestions

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 28 11:01:18 PST 2013


Enrica Casucci <enrica at apple.com> has granted Grzegorz Czajkowski
<g.czajkowski at samsung.com>'s request for review:
Bug 109577: [WK2] Asynchronous spell checking implementation
https://bugs.webkit.org/show_bug.cgi?id=109577

Attachment 190027: apply Anders' suggestions
https://bugs.webkit.org/attachment.cgi?id=190027&action=review

------- Additional Comments from Enrica Casucci <enrica at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=190027&action=review


Overall it looks good to me. Just some minor comments about the changelog.

> Source/WebKit2/ChangeLog:28
> +	   (CoreIPC::::decode):

Remove the 2 lines above.

> Source/WebKit2/ChangeLog:36
> +	   (TextChecker):

Remove.

> Source/WebKit2/ChangeLog:40
> +	   (WebKit):

Ditto.

> Source/WebKit2/ChangeLog:58
> +	   (TextCheckerCompletion):

ditto

> Source/WebKit2/ChangeLog:120
> +	   (WebPage):

Ditto.


More information about the webkit-reviews mailing list