[Webkit-unassigned] [Bug 63050] [chromium] cannot replace a misspelled phrase with a suggested word

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=63050





--- Comment #3 from MORITA Hajime <morrita at google.com>  2011-06-21 23:57:34 PST ---
(From update of attachment 97951)
View in context: https://bugs.webkit.org/attachment.cgi?id=97951&action=review

> Source/WebKit/chromium/src/WebFrameImpl.cpp:1148
>  }

As you said, I believe this is worth having inside WebCore.
If we have this inside WebCore, we no longer need new TextInputController method.
Also, this behavior is desirable for spelling marker in general, not only for API-based spellchecking.

> LayoutTests/editing/spelling/spellcheck-api-replace.html:7
> +<script language="javascript" type="text/javascript">

Could you get these two <script> block together?

> LayoutTests/editing/spelling/spellcheck-api-replace.html:18
> +function RunTest() {

we usually use camel case name like "runTest"

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.