[Webkit-unassigned] [Bug 37050] Toggle spell check doesn't update markers

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 2 17:39:09 PDT 2010


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


Tony Chang (Google) <tony at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tony at chromium.org




--- Comment #1 from Tony Chang (Google) <tony at chromium.org>  2010-06-02 17:39:08 PST ---
(In reply to comment #0)
> I propose that in Editor.cpp:toggleContinuousSpellChecking(), the markers should be updated after toggling. I'm interested in putting a patch together which does this and adds a LayoutTest that ensures markers are updated after toggling on and off.

This sounds ok to me-- maybe you can use markAllMisspellingsInRange.  The main thing to test is how this performs on very large documents.  I'm a bit worried it might lock up the UI.

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



More information about the webkit-unassigned mailing list