Make default spellchecking behavior in html elements configurable via WebSettings.

third_party/WebKit/Source/core/dom/Element.cpp

Index: third_party/WebKit/Source/core/dom/Element.cpp
diff --git a/third_party/WebKit/Source/core/dom/Element.cpp b/third_party/WebKit/Source/core/dom/Element.cpp
index 1e9ac996a103133d46428a7536e41ccbf27841a7..6db8255ae7226f7ba3fbcc2564da96e11ee86e1a 100644
--- a/third_party/WebKit/Source/core/dom/Element.cpp
+++ b/third_party/WebKit/Source/core/dom/Element.cpp
@@ -3208,7 +3208,11 @@ bool Element::isSpellCheckingEnabled() const
         }
     }
 
+#if OS(ANDROID)
+    return false;

[Peter Beverloo, 2016/09/12 14:59:02]

This should probably be a decision made by the embedder, i.e. routed through
WebSettings?

[Rick Byers, 2016/09/14 15:21:00]

Yes, we avoid adding #ifdefs like this in Blink.  Eg. it lets us write tests for
both modes that run on all platforms rather than relying on running the tests on
Android (which we still don't do for many tests).

[timvolodine, 2016/09/14 17:16:31]

ok, sounds right, thanks for the suggestion Peter!
I've now implemented it via settings.

+#else
     return true;
+#endif
 }
 
 #if DCHECK_IS_ON()