Use improved VariationParamsManager to hide details.

components/ntp_snippets/remote/ntp_snippets_fetcher.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_NTP_SNIPPETS_REMOTE_NTP_SNIPPETS_FETCHER_H_
 #define COMPONENTS_NTP_SNIPPETS_REMOTE_NTP_SNIPPETS_FETCHER_H_
 
 #include <memory>
 #include <queue>
 #include <set>
@@ skipping 212 matching lines @@
         const scoped_refptr<net::URLRequestContextGetter>& context_getter);
     RequestBuilder& SetUserClassifier(const UserClassifier& user_classifier);
 
     std::string PreviewRequestBodyForTesting() { return BuildBody(); }
     std::string PreviewRequestHeadersForTesting() { return BuildHeaders(); }
     RequestBuilder& SetUserClassForTesting(const std::string& user_class) {
       user_class_ = user_class;
       return *this;
     }
 
-   protected:
-    // TODO(fhorschig): As soon as crbug.com/crbug.com/645447 is resolved,
-    // make these an implementation detail in the body and remove the mock.
-    virtual bool IsSendingTopLanguagesEnabled() const;
-    virtual bool IsSendingUserClassEnabled() const;
-
    private:
     std::string BuildHeaders() const;
     std::string BuildBody() const;
     std::unique_ptr<net::URLFetcher> BuildURLFetcher(
         net::URLFetcherDelegate* request,
         const std::string& headers,
         const std::string& body) const;
 
     bool ReturnOnlyPersonalizedResults() const {
       return !obfuscated_gaia_id_.empty() &&
@@ skipping 108 matching lines @@
   std::string last_status_;
   std::string last_fetch_json_;
 
   base::WeakPtrFactory<NTPSnippetsFetcher> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(NTPSnippetsFetcher);
 };
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_REMOTE_NTP_SNIPPETS_FETCHER_H_