[spell] Remove spelling service feedback from behind the flag

chrome/browser/spellchecker/feedback_sender.h

 // 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.
 //
 // An object to record and send user feedback to spelling service. The spelling
 // service uses the feedback to improve its suggestions.
 //
 // Assigns uint32 hash identifiers to spelling suggestions from spelling service
 // and stores these suggestions. Records user's actions on these suggestions.
 // Periodically sends batches of user feedback to the spelling service.
@@ skipping 85 matching lines @@
   void OnSpellcheckResults(int renderer_process_id,
                            const string16& text,
                            const std::vector<SpellCheckMarker>& markers,
                            std::vector<SpellCheckResult>* results);
 
   // Receives updated language and country code for feedback. Finalizes and
   // sends out all of the feedback data.
   void OnLanguageCountryChange(const std::string& language,
                                const std::string& country);
 
+  // Stops collecting feedback. Sends out all previously collected data.
+  void StopFeedbackCollection();

[groby-ooo-7-16, 2013/11/21 02:48:41]

Please put Start before Stop - makes sense both alphabetically and logically :)

+
+  // Starts collecting feedback.
+  void StartFeedbackCollection();
+
  private:
   friend class FeedbackSenderTest;
 
   // net::URLFetcherDelegate implementation. Takes ownership of |source|.
   virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
 
   // Requests the document markers from all of the renderers to determine which
   // feedback can be finalized. Finalizes feedback for renderers that are gone.
   // Called periodically when |timer_| fires.
   void RequestDocumentMarkers();
 
   // Sends out all feedback data. Resets the session-start timestamp to now.
   // Restarts the timer that requests markers from the renderers.
   void FlushFeedback();
 
   // Sends out the |feedback_data|.
   void SendFeedback(const std::vector<Misspelling>& feedback_data,
                     bool is_first_feedback_batch);
 
   // URL request context for the feedback senders.
   scoped_refptr<net::URLRequestContextGetter> request_context_;
 
+  // The feedback API version.
+  const std::string api_version_;
+
   // The language of text. The string is a BCP 47 language tag.
   std::string language_;
 
   // The country of origin. The string is the ISO 3166-1 alpha-3 code.
   std::string country_;
 
   // Misspelling counter used to generate unique hash identifier for each
   // misspelling.
   size_t misspelling_counter_;
 
@@ skipping 19 matching lines @@
   // If a sender is destroyed before it finishes, then sending feedback will be
   // canceled.
   ScopedVector<net::URLFetcher> senders_;
 
   DISALLOW_COPY_AND_ASSIGN(FeedbackSender);
 };
 
 }  // namespace spellcheck
 
 #endif  // CHROME_BROWSER_SPELLCHECKER_FEEDBACK_SENDER_H_