Add a switch to TestRunner to enable/disable mock spell checker

components/test_runner/spell_check_client.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/test_runner/spell_check_client.h"
 
 #include <stddef.h>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "components/test_runner/mock_grammar_check.h"
 #include "components/test_runner/test_runner.h"
 #include "components/test_runner/web_task.h"
 #include "components/test_runner/web_test_delegate.h"
 #include "third_party/WebKit/public/web/WebTextCheckingCompletion.h"
 #include "third_party/WebKit/public/web/WebTextCheckingResult.h"
 
 namespace test_runner {
 
 SpellCheckClient::SpellCheckClient(TestRunner* test_runner)
-    : last_requested_text_checking_completion_(nullptr),
+    : enabled_(false),

[yosin_UTC9, 2016/08/24 08:00:38]

nit: let's initialize |enabled_| in class declaration.

[Xiaocheng, 2016/08/24 08:28:23]

Done.

+      last_requested_text_checking_completion_(nullptr),
       test_runner_(test_runner),
       weak_factory_(this) {
   DCHECK(test_runner);
 }
 
 SpellCheckClient::~SpellCheckClient() {
 }
 
 void SpellCheckClient::SetDelegate(WebTestDelegate* delegate) {
   delegate_ = delegate;
 }
 
+void SpellCheckClient::SetEnabled(bool enabled) {
+  enabled_ = enabled;
+}
+
 // blink::WebSpellCheckClient
 void SpellCheckClient::spellCheck(
     const blink::WebString& text,
     int& misspelled_offset,
     int& misspelled_length,
     blink::WebVector<blink::WebString>* optional_suggestions) {
+  if (!enabled_) {
+    misspelled_offset = 0;
+    misspelled_length = 0;
+    return;
+  }
+
   // Check the spelling of the given text.
   spell_check_.SpellCheckWord(text, &misspelled_offset, &misspelled_length);
 }
 
 void SpellCheckClient::requestCheckingOfText(
     const blink::WebString& text,
     const blink::WebVector<uint32_t>& markers,
     const blink::WebVector<unsigned>& marker_offsets,
     blink::WebTextCheckingCompletion* completion) {
-  if (text.isEmpty()) {
+  if (!enabled_ || text.isEmpty()) {
     if (completion)
       completion->didCancelCheckingText();
     return;
   }
 
   if (last_requested_text_checking_completion_)
     last_requested_text_checking_completion_->didCancelCheckingText();
 
   last_requested_text_checking_completion_ = completion;
   last_requested_text_check_string_ = text;
@@ skipping 43 matching lines @@
                                            &results);
   }
   last_requested_text_checking_completion_->didFinishCheckingText(results);
   last_requested_text_checking_completion_ = 0;
 
   if (test_runner_->shouldDumpSpellCheckCallbacks())
     delegate_->PrintMessage("SpellCheckEvent: FinishLastTextCheck\n");
 }
 
 }  // namespace test_runner