[Webkit-unassigned] [Bug 66812] [Qt] Spell check platform plugin API enhancement

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Dawit A. <adawit at kde.org>  2011-08-30 12:03:02 PST ---
(In reply to comment #2)
> (From update of attachment 104917 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=104917&action=review
> 
> > Source/WebKit/qt/WebCoreSupport/TextCheckerClientQt.cpp:55
> > -    m_spellChecker->ignoreWordInSpellDocument(word);
> > +    m_spellChecker->ignoreWord(word);
> 
> I think a mapping 1 to 1 between the WebKit APIs and the platform plugin is a good thing. Platform plugins are a low level tool for us, it does not needs a beautiful API, it needs a totally unambiguous API.

Ok. Will revert.

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