Chromium Code Reviews| Index: components/ntp_snippets/ntp_snippets_fetcher.h |
| diff --git a/components/ntp_snippets/ntp_snippets_fetcher.h b/components/ntp_snippets/ntp_snippets_fetcher.h |
| index 533f557d4981371ece78766c01950792cede9387..3c1a2608bbd0e4daa1bb0938e516752536c78464 100644 |
| --- a/components/ntp_snippets/ntp_snippets_fetcher.h |
| +++ b/components/ntp_snippets/ntp_snippets_fetcher.h |
| @@ -99,7 +99,7 @@ class NTPSnippetsFetcher : public OAuth2TokenService::Consumer, |
| } |
| 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 meaningf
jkrcal
2016/05/12 09:57:29
Done. I did not like kYes, kNo, kBoth - Yes and No
Marc Treib
2016/05/12 10:06:15
Yup, yours are better, thanks!
|
| void FetchSnippetsImpl(const GURL& url, |
| const std::string& auth_header, |
| @@ -163,8 +163,10 @@ class NTPSnippetsFetcher : public OAuth2TokenService::Consumer, |
| // 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_; |