Make SpellChecker::clearMisspellingsForMovingParagraphs() to take SelectionInDOMTree

third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp

Index: third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
diff --git a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
index 1bb7922fe8f2ec84519561467181a43c7e058540..d96c26dc7134258a9d284a9003c3f1713db1af96 100644
--- a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
+++ b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
@@ -319,9 +319,10 @@ void SpellChecker::showSpellingGuessPanel() {
 }
 
 void SpellChecker::clearMisspellingsForMovingParagraphs(
-    const VisibleSelection& movingSelection) {
-  removeMarkers(movingSelection.toNormalizedEphemeralRange(),
-                DocumentMarker::MisspellingMarkers());
+    const SelectionInDOMTree& movingSelection) {
+  removeMarkers(
+      createVisibleSelection(movingSelection).toNormalizedEphemeralRange(),

[Xiaocheng, 2017/03/09 19:14:23]

Is there any way to get rid of this canonicalization?

[yosin_UTC9, 2017/03/10 00:58:00]

I guess we can use VS::start/end here.
Or, we get rid of this function at all.
Caller remove selection passed this function after this call.
Markers should remove by node removal.

I file an issue crbug.com/700217 We should get rid of
SpellChecker::clearMisspellingsForMovingParagraphs()

+      DocumentMarker::MisspellingMarkers());
 }
 
 void SpellChecker::markMisspellingsForMovingParagraphs(