Allow getting _only_ personalized snippets for NTP.

components/ntp_snippets/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_NTP_SNIPPETS_FETCHER_H_
 #define COMPONENTS_NTP_SNIPPETS_NTP_SNIPPETS_FETCHER_H_
 
 #include <memory>
 #include <set>
 #include <string>
@@ skipping 81 matching lines @@
   const std::string& last_json() const {
     return last_fetch_json_;
   }
 
   // Overrides internal clock for testing purposes.
   void SetTickClockForTesting(std::unique_ptr<base::TickClock> tick_clock) {
     tick_clock_ = std::move(tick_clock);
   }
 
  private:
-  enum class Variant { kRestrictedPersonalized, kRestricted, kPersonalized };
+  enum class Variant { kPersonalized, kNonPersonalized, kAll };

[Marc Treib, 2016/05/12 08:59:16]

Can we rename "Variant" to something more meaningful now? "Personalization" with
values kYes, kNo, kBoth, something like that?

[jkrcal, 2016/05/12 09:57:29]

Done. I did not like kYes, kNo, kBoth - Yes and No sound too exclusive to be
combined into Both.

[Marc Treib, 2016/05/12 10:06:15]

Yup, yours are better, thanks!

 
   void FetchSnippetsImpl(const GURL& url,
                          const std::string& auth_header,
                          const std::string& request);
   std::string GetHostRestricts() const;
   bool UseHostRestriction() const;
   bool UseAuthentication() const;
   void FetchSnippetsNonAuthenticated();
   void FetchSnippetsAuthenticated(const std::string& account_id,
                                   const std::string& oauth_access_token);
@@ skipping 43 matching lines @@
 
   // The fetcher for downloading the snippets.
   std::unique_ptr<net::URLFetcher> url_fetcher_;
 
   // The callback to notify when new snippets get fetched.
   SnippetsAvailableCallback snippets_available_callback_;
 
   // Flag for picking the right (stable/non-stable) API key for Chrome Reader.
   bool is_stable_channel_;
 
-  // The variant of the fetching to use.
+  // The variant of the fetching to use, loaded from variation parameters.
   Variant variant_;
+  // Should we apply host restriction? It is loaded from variation parameters.
+  bool use_host_restriction_;
 
   // Allow for an injectable tick clock for testing.
   std::unique_ptr<base::TickClock> tick_clock_;
 
   base::WeakPtrFactory<NTPSnippetsFetcher> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(NTPSnippetsFetcher);
 };
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_NTP_SNIPPETS_FETCHER_H_