[webkit-reviews] review granted: [Bug 95139] [BlackBerry] Spellcheck should be enabled if undefined in content : [Attachment 160974] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Aug 28 08:14:34 PDT 2012


Rob Buis <rwlbuis at gmail.com> has granted Nima Ghanavatian
<nima.ghanavatian at gmail.com>'s request for review:
Bug 95139: [BlackBerry] Spellcheck should be enabled if undefined in content
https://bugs.webkit.org/show_bug.cgi?id=95139

Attachment 160974: Patch
https://bugs.webkit.org/attachment.cgi?id=160974&action=review

------- Additional Comments from Rob Buis <rwlbuis at gmail.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=160974&action=review


Looks good.

> Source/WebKit/blackberry/WebKitSupport/InputHandler.h:202
> +    bool shouldSpellCheckElement(const WebCore::Element*);

Can the method be const?


More information about the webkit-reviews mailing list