Fix bug where misspelling underline is not removed properly

third_party/WebKit/Source/web/tests/WebFrameTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 6631 matching lines @@
 
   EXPECT_EQ(1, textcheck.NumberOfTimesChecked());
   EXPECT_EQ(1, NumMarkersInRange(document, selection_range,
                                  DocumentMarker::kSpelling));
 
   frame->ReplaceMisspelledRange("welcome");
   EXPECT_EQ("_welcome_.",
             WebFrameContentDumper::DumpWebViewAsText(
                 web_view_helper.WebView(), std::numeric_limits<size_t>::max())
                 .Utf8());
+
+  const int kAllNewTextBeginOffset = 0;
+  const int kAllNewTextLength = 10;
+  frame->SelectRange(WebRange(kAllNewTextBeginOffset, kAllNewTextLength));
+  selection_range = frame->GetFrame()
+                        ->Selection()
+                        .ComputeVisibleSelectionInDOMTreeDeprecated()
+                        .ToNormalizedEphemeralRange();
+  EXPECT_EQ(0, NumMarkersInRange(document, selection_range,
+                                 DocumentMarker::kSpelling));
 }
 
 TEST_P(ParameterizedWebFrameTest, RemoveSpellingMarkers) {
   RegisterMockedHttpURLLoad("spell.html");
   FrameTestHelpers::WebViewHelper web_view_helper;
   InitializeTextSelectionWebView(base_url_ + "spell.html", &web_view_helper);
 
   WebLocalFrameImpl* frame = web_view_helper.WebView()->MainFrameImpl();
   TextCheckClient textcheck;
   frame->SetTextCheckClient(&textcheck);
@@ skipping 5485 matching lines @@
 
   // Failing the original child frame navigation and trying to render fallback
   // content shouldn't crash. It should return NoLoadInProgress. This is so the
   // caller won't attempt to replace the correctly empty frame with an error
   // page.
   EXPECT_EQ(WebLocalFrame::NoLoadInProgress,
             child->MaybeRenderFallbackContent(WebURLError()));
 }
 
 }  // namespace blink