Adding a word to dictionary should remove spelling markers

Source/core/editing/SpellChecker.cpp

Index: Source/core/editing/SpellChecker.cpp
diff --git a/Source/core/editing/SpellChecker.cpp b/Source/core/editing/SpellChecker.cpp
index fb57b8b84a4eb81d8e87258c962b7c053072a98d..c1180918d77fa3b534d11e9b172e59a655e66060 100644
--- a/Source/core/editing/SpellChecker.cpp
+++ b/Source/core/editing/SpellChecker.cpp
@@ -824,6 +824,27 @@ void SpellChecker::removeSpellingMarkers()
     m_frame.document()->markers().removeMarkers(DocumentMarker::MisspellingMarkers());
 }
 
+void SpellChecker::removeSpellingMarkersUnderWords(const Vector<String> &words)
+{

[groby-ooo-7-16, 2014/07/30 21:13:27]

You might want to test this for large documents. IIRC, whole-document checking
was definitely too slow. It might be that removal is fine, since it doesn't
involve any IPC, but it's worth trying out.

[Klemen Forstnerič, 2014/07/31 17:14:00]

Could you point out which piece of code was shown to be the bottleneck?

[groby-ooo-7-16, 2014/08/01 00:13:22]

The bottleneck for checking was the IPC traffic. You don't have that here, but
you still traverse the entire document. I honestly have no idea how much impact
that has - I'd try something like taking a giant text in a non-english language
(Proust comes to mind :), pasting it into the text field, adding one word to the
dictionary, and just checking if there's perceptible lag.

I'm not super-worried - it'll likely be fine - but it's worth checking.

[Klemen Forstnerič, 2014/08/01 21:58:17]

Seems to work fast enough. :-)

+    auto document = m_frame.document();
+    auto& markerController = document->markers();
+
+    auto markerNodePairs = markerController.markerNodePairs();
+    for (const auto &markerNodePair: markerNodePairs) {
+        auto node = const_cast<Node *>(markerNodePair.first.get());
+        auto marker = markerNodePair.second;
+
+        auto markerRange = Range::create(*document,
+            node, marker->startOffset(),
+            node, marker->endOffset());
+
+        if (words.contains(markerRange->text()))
+            markerController.removeMarkers(markerRange.get());
+    }
+
+    markerController.repaintMarkers();
+}
+
 void SpellChecker::spellCheckAfterBlur()
 {
     if (!m_frame.selection().selection().isContentEditable())