[webkit-reviews] review canceled: [Bug 59693] [Feature Request] Need SpellCheck API : [Attachment 94756] Patch v1 (only for Chromium)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 25 04:38:32 PDT 2011


Hironori Bono <hbono at chromium.org> has canceled Hironori Bono
<hbono at chromium.org>'s request for review:
Bug 59693: [Feature Request] Need SpellCheck API
https://bugs.webkit.org/show_bug.cgi?id=59693

Attachment 94756: Patch v1 (only for Chromium)
https://bugs.webkit.org/attachment.cgi?id=94756&action=review

------- Additional Comments from Hironori Bono <hbono at chromium.org>
Greetings,

(In reply to comment #6)
> Source/WebCore/html/HTMLTextAreaElement.cpp:55:  Alphabetical sorting
problem.  [build/include_order] [4]
> Source/WebCore/html/HTMLInputElement.cpp:64:	Alphabetical sorting problem. 
[build/include_order] [4]
> Source/WebCore/html/HTMLDivElement.cpp:33:  Alphabetical sorting problem. 
[build/include_order] [4]
> Source/WebCore/dom/DocumentMarkerController.h:85:  The parameter name "node"
adds no information, so it should be removed.  [readability/parameter_name] [5]

> Total errors found: 4 in 30 files

Oops, I forgot running check-webkit-style. I have updated my change.

Regards,

Hironori Bono


More information about the webkit-reviews mailing list