Componentize spellcheck [1]: create component and move switches there.

chrome/browser/spellchecker/feedback_sender.cc

 // Copyright (c) 2013 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.
 //
 // The |FeedbackSender| object stores the user feedback to spellcheck
 // suggestions in a |Feedback| object.
 //
 // When spelling service returns spellcheck results, these results first arrive
 // in |FeedbackSender| to assign hash identifiers for each
 // misspelling-suggestion pair. If the spelling service identifies the same
@@ skipping 27 matching lines @@
 #include "base/json/json_writer.h"
 #include "base/location.h"
 #include "base/metrics/field_trial.h"
 #include "base/single_thread_task_runner.h"
 #include "base/stl_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/values.h"
 #include "chrome/browser/spellchecker/word_trimmer.h"
-#include "chrome/common/chrome_switches.h"
 #include "chrome/common/spellcheck_common.h"
 #include "chrome/common/spellcheck_marker.h"
 #include "chrome/common/spellcheck_messages.h"
 #include "components/data_use_measurement/core/data_use_user_data.h"
+#include "components/spellcheck/common/spellcheck_switches.h"
 #include "content/public/browser/render_process_host.h"
 #include "crypto/random.h"
 #include "crypto/secure_hash.h"
 #include "crypto/sha2.h"
 #include "google_apis/google_api_keys.h"
 #include "net/base/load_flags.h"
 #include "net/url_request/url_fetcher.h"
 #include "net/url_request/url_request_context_getter.h"
 
 namespace spellcheck {
@@ skipping 113 matching lines @@
              text_length;
 }
 
 // Returns the feedback API version.
 std::string GetApiVersion() {
   // This guard is temporary.
   // TODO(rouslan): Remove the guard. http://crbug.com/247726
   if (base::FieldTrialList::FindFullName(kFeedbackFieldTrialName) ==
           kFeedbackFieldTrialEnabledGroupName &&
       base::CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kEnableSpellingFeedbackFieldTrial)) {
+          spellcheck::switches::kEnableSpellingFeedbackFieldTrial)) {
     return "v2-internal";
   }
   return "v2";
 }
 
 }  // namespace
 
 FeedbackSender::FeedbackSender(net::URLRequestContextGetter* request_context,
                                const std::string& language,
                                const std::string& country)
     : request_context_(request_context),
       api_version_(GetApiVersion()),
       language_(language),
       country_(country),
       misspelling_counter_(0),
       feedback_(kMaxFeedbackSizeBytes),
       session_start_(base::Time::Now()),
       feedback_service_url_(kFeedbackServiceURL) {
   // The command-line switch is for testing and temporary.
   // TODO(rouslan): Remove the command-line switch when testing is complete.
   // http://crbug.com/247726
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kSpellingServiceFeedbackUrl)) {
+          spellcheck::switches::kSpellingServiceFeedbackUrl)) {
     feedback_service_url_ =
         GURL(base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
-            switches::kSpellingServiceFeedbackUrl));
+            spellcheck::switches::kSpellingServiceFeedbackUrl));
   }
 }
 
 FeedbackSender::~FeedbackSender() {
 }
 
 void FeedbackSender::SelectedSuggestion(uint32_t hash, int suggestion_index) {
   Misspelling* misspelling = feedback_.GetMisspelling(hash);
   // GetMisspelling() returns null for flushed feedback. Feedback is flushed
   // when the session expires every |kSessionHours| hours.
@@ skipping 118 matching lines @@
 
 void FeedbackSender::StartFeedbackCollection() {
   if (timer_.IsRunning())
     return;
 
   int interval_seconds = chrome::spellcheck_common::kFeedbackIntervalSeconds;
   // This command-line switch is for testing and temporary.
   // TODO(rouslan): Remove the command-line switch when testing is complete.
   // http://crbug.com/247726
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kSpellingServiceFeedbackIntervalSeconds)) {
+          spellcheck::switches::kSpellingServiceFeedbackIntervalSeconds)) {
     base::StringToInt(
         base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
-            switches::kSpellingServiceFeedbackIntervalSeconds),
+            spellcheck::switches::kSpellingServiceFeedbackIntervalSeconds),
         &interval_seconds);
     if (interval_seconds < kMinIntervalSeconds)
       interval_seconds = kMinIntervalSeconds;
     static const int kSessionSeconds =
         chrome::spellcheck_common::kSessionHours * 60 * 60;
     if (interval_seconds >  kSessionSeconds)
       interval_seconds = kSessionSeconds;
   }
   timer_.Start(FROM_HERE,
                base::TimeDelta::FromSeconds(interval_seconds),
@@ skipping 90 matching lines @@
   senders_.push_back(sender);
 
   // Request context is nullptr in testing.
   if (request_context_.get()) {
     sender->SetRequestContext(request_context_.get());
     sender->Start();
   }
 }
 
 }  // namespace spellcheck