[webkit-reviews] review requested: [Bug 59693] [Feature Request] Need SpellCheck API : [Attachment 102119] Patch v11

[bugzilla-daemon at webkit.org]

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

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

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

Thank you for your comments and sorry for my slow response.

(In reply to comment #36)
> Where is the spec for this feature? I can't find it. Your Changelog entry
should reference it.

Thank you for noticing it. Even though we have talked about this API last
month, it is not so easy to read though the thread. I have sent an updated spec
to summarize the thread
<http://lists.w3.org/Archives/Public/public-webapps/2011JulSep/0413.html> and
added this URL to ChangeLog.

(In reply to comment #37)
> (From update of attachment 98897 [details])
> Has this feature had a security review?

Thanks to your help, we have finally had a change to talk with them about this
API. If I understand the talk with our security people correctly, they do not
block this API. (It would be great to correct me if I'm wrong.)

(In reply to comment #39)
> Shouldn't this be WebKitSpelllcheckRange?
> WebKitSpellcheckRangeList?

You are right. I totally forgot renaming them. I have updated this change to
rename them.

Regards,

Hironori Bono