Adding a word to dictionary should remove spelling markers

Source/web/WebViewImpl.cpp

Index: Source/web/WebViewImpl.cpp
diff --git a/Source/web/WebViewImpl.cpp b/Source/web/WebViewImpl.cpp
index 1b3aa3ed31d52c0d60fd5fcf93ad07499d51043f..f0ac81dafd58b8da78a8df7f0920a3cd4f374625 100644
--- a/Source/web/WebViewImpl.cpp
+++ b/Source/web/WebViewImpl.cpp
@@ -3421,6 +3421,19 @@ void WebViewImpl::spellingMarkers(WebVector<uint32_t>* markers)
     markers->assign(result);
 }
 
+void WebViewImpl::removeSpellingMarkersUnderWords(const WebVector<WebString>& words)
+{
+    for (Frame* frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
+        if (!frame->isLocalFrame())
+            continue;
+
+        Vector<String> convertedWords;
+        convertedWords.append(words.data(), words.size());

[please use gerrit instead, 2014/08/15 19:08:07]

Although the number of frames may be slow, it's best to avoid creating the same
vector inside of a loop. (Note that frame->removeSpellingMarkersUnderWords()
does not alter this vector.) Please move the vector creation outside of the
loop.

[Klemen Forstnerič, 2014/08/16 15:45:50]

Done.

+
+        toLocalFrame(frame)->removeSpellingMarkersUnderWords(convertedWords);
+    }
+}
+
 WebDragOperation WebViewImpl::dragTargetDragEnterOrOver(const WebPoint& clientPoint, const WebPoint& screenPoint, DragAction dragAction, int keyModifiers)
 {
     ASSERT(m_currentDragData);