Inject API key directly into snippets fetcher.

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 66 matching lines @@
     kBoth
   };
 
   NTPSnippetsFetcher(
       SigninManagerBase* signin_manager,
       OAuth2TokenService* oauth2_token_service,
       scoped_refptr<net::URLRequestContextGetter> url_request_context_getter,
       PrefService* pref_service,
       CategoryFactory* category_factory,
       const ParseJSONCallback& parse_json_callback,
-      bool is_stable_channel);
+      const std::string& api_key);
   ~NTPSnippetsFetcher() override;
 
   // Set a callback that is called when a new set of snippets are downloaded,
   // overriding any previously set callback.
   void SetCallback(const SnippetsAvailableCallback& callback);
 
   // Fetches snippets from the server. |hosts| restricts the results to a set of
   // hosts, e.g. "www.google.com". If host restrictions are enabled, an empty
   // host set produces an error without issuing a fetch.
   //
@@ skipping 126 matching lines @@
   const GURL fetch_url_;
   // Which API to use
   const FetchAPI fetch_api_;
 
   // 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.

[Marc Treib, 2016/08/31 09:26:16]

Please update the comment

[sfiera, 2016/08/31 10:09:16]

Done.

-  bool is_stable_channel_;
+  const std::string api_key_;
 
   // The variant of the fetching to use, loaded from variation parameters.
   Personalization personalization_;
   // Should we apply host restriction? It is loaded from variation parameters.
   bool use_host_restriction_;
 
   // Is the request user initiated?
   bool interactive_request_;
 
   // Allow for an injectable tick clock for testing.
   std::unique_ptr<base::TickClock> tick_clock_;
 
   // Request throttler for limiting requests.
   RequestThrottler request_throttler_;
 
   // When a token request gets canceled, we want to retry once.
   bool oauth_token_retried_;
 
   base::WeakPtrFactory<NTPSnippetsFetcher> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(NTPSnippetsFetcher);
 };
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_NTP_SNIPPETS_FETCHER_H_