Fix bug where misspelling underline is not removed properly

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 bd6fad960936c59028da2e15856885b717236563..6a800400a5973496bfc3a1533981b7f877286b49 100644
--- a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
+++ b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
@@ -838,6 +838,9 @@ void SpellChecker::ReplaceMisspelledRange(const String& text) {
   if (marker_range.IsNull())
     return;
 
+  GetFrame().GetDocument()->Markers().RemoveMarkersInRange(

[yosin_UTC9, 2017/05/22 05:14:10]

If we decide to remove marker explicitly, I against,
It should be done just before, or after(?), 
| GetFrame().GetEditor().ReplaceSelectionWithText()| call
to respect "beforeinput" event, at L853.

+      marker_range, DocumentMarker::MisspellingMarkers());

[yosin_UTC9, 2017/05/22 05:14:10]

I'm not sure why do we need to remove marker explicitly?

We should remove other types of markers too, e.g. text match marker.

Editor::ReplaceSelectionWithText(), which is called at end of
|SpellChecker::ReplaceMisspelledRange()|, should remove a markers.

+
   GetFrame().Selection().SetSelection(
       SelectionInDOMTree::Builder().SetBaseAndExtent(marker_range).Build());