Fix bug where misspelling underline is not removed properly

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

 /*
  * Copyright (C) 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 820 matching lines @@
   if (marker_it == markers_in_node.end())
     return;
 
   const DocumentMarker* found_marker = *marker_it;
   EphemeralRange marker_range = EphemeralRange(
       Position(caret_start_container, found_marker->StartOffset()),
       Position(caret_start_container, found_marker->EndOffset()));
   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());
 
   Document& current_document = *GetFrame().GetDocument();
 
   // Dispatch 'beforeinput'.
   Element* const target = GetFrame().GetEditor().FindEventTargetFromSelection();
   DataTransfer* const data_transfer = DataTransfer::Create(
       DataTransfer::DataTransferType::kInsertReplacementText,
       DataTransferAccessPolicy::kDataTransferReadable,
@@ skipping 369 matching lines @@
       if (!input.IsFocusedElementInDocument())
         return false;
     }
   }
   HTMLElement* element =
       Traversal<HTMLElement>::FirstAncestorOrSelf(*position.AnchorNode());
   return element && element->IsSpellCheckingEnabled();
 }
 
 }  // namespace blink