Remove spelling markers in disabled and readonly form elements

third_party/WebKit/Source/core/html/HTMLElement.cpp

Index: third_party/WebKit/Source/core/html/HTMLElement.cpp
diff --git a/third_party/WebKit/Source/core/html/HTMLElement.cpp b/third_party/WebKit/Source/core/html/HTMLElement.cpp
index f032cf0e675232a03d5e1cf8cd1ca893be0aed15..0631596c6fbddbc193d14bfe86c7f4038ab2efe9 100644
--- a/third_party/WebKit/Source/core/html/HTMLElement.cpp
+++ b/third_party/WebKit/Source/core/html/HTMLElement.cpp
@@ -430,10 +430,8 @@ void HTMLElement::attributeChanged(const AttributeModificationParams& params) {
     if (adjustedFocusedElementInTreeScope() == this)
       blur();
   } else if (params.name == contenteditableAttr) {
-    if (document().frame()) {
-      document().frame()->spellChecker().removeSpellingAndGrammarMarkers(
-          *this, SpellChecker::ElementsType::kOnlyNonEditable);
-    }
+    if (document().frame())
+      document().frame()->spellChecker().cancelCheck();

[Xiaocheng, 2017/03/03 18:24:06]

SpellChecker::cancelCheck is not the right function to call; it simply cancels
the pending spell check request, if any. For example, if we:

1. Paste something into an editable, which triggers a spell check request
2. Before the request is resolved, set another editable as non-editable

This patch makes step 2 call SpellChecker::cancelCheck, which simply cancels the
checking for the first editable

     if (adjustedFocusedElementInTreeScope() != this)
       return;
     // The attribute change may cause supportsFocus() to return false