Adding a word to dictionary should remove spelling markers

Source/core/dom/DocumentMarkerController.cpp

Index: Source/core/dom/DocumentMarkerController.cpp
diff --git a/Source/core/dom/DocumentMarkerController.cpp b/Source/core/dom/DocumentMarkerController.cpp
index 5a7cc9b47afb89e7f4f28bc2d60120995ec16109..a14c915a0cd8f9eb2a91f70dbcd0758c01ccb0ae 100644
--- a/Source/core/dom/DocumentMarkerController.cpp
+++ b/Source/core/dom/DocumentMarkerController.cpp
@@ -417,6 +417,20 @@ DocumentMarkerVector DocumentMarkerController::markers()
     return result;
 }
 
+DocumentMarkerAndNodeVector DocumentMarkerController::markerNodePairs()

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

This seems to copy a lot of data, depending on the amount of markers. Why not a
removeMarker that takes a predicate instead?

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

That seems like a great idea, I'll add a removeMarkers method with a predicate.

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

Acknowledged.

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

Done.

+{
+    DocumentMarkerAndNodeVector result;
+    for (MarkerMap::iterator i = m_markers.begin(); i != m_markers.end(); ++i) {
+        MarkerLists* markers = i->value.get();
+        for (size_t markerListIndex = 0; markerListIndex < DocumentMarker::MarkerTypeIndexesCount; ++markerListIndex) {
+            OwnPtrWillBeMember<MarkerList>& list = (*markers)[markerListIndex];
+            for (size_t j = 0; list.get() && j < list->size(); ++j)
+                result.append(std::make_pair(i->key, list->at(j).get()));
+        }
+    }
+    return result;
+}
+
 DocumentMarkerVector DocumentMarkerController::markersInRange(Range* range, DocumentMarker::MarkerTypes markerTypes)
 {
     if (!possiblyHasMarkers(markerTypes))