Adding a parameter that will be used to request a transparent doodle.

components/search_provider_logos/logo_tracker.h

Index: components/search_provider_logos/logo_tracker.h
diff --git a/components/search_provider_logos/logo_tracker.h b/components/search_provider_logos/logo_tracker.h
index 324d86a07d3e714d801fd16e5ec00b2003479439..722a9ddc60d2dc38f4f8c2fa14d40347d9f9222c 100644
--- a/components/search_provider_logos/logo_tracker.h
+++ b/components/search_provider_logos/logo_tracker.h
@@ -79,7 +79,8 @@ typedef base::Callback<std::unique_ptr<EncodedLogo>(
 // server to verify whether the cached logo is up-to-date.
 typedef base::Callback<GURL(const GURL& logo_url,
                             const std::string& fingerprint,
-                            bool wants_cta)> AppendQueryparamsToLogoURL;
+                            bool wants_cta,
+                            bool transparent)> AppendQueryparamsToLogoURL;
 
 // This class provides the logo for a search provider. Logos are downloaded from
 // the search provider's logo URL and cached on disk.
@@ -123,13 +124,15 @@ class LogoTracker : public net::URLFetcherDelegate {
   // server's response into a EncodedLogo object. |append_queryparams_func| is a
   // callback that will return the URL from which to download the logo.
   // |wants_cta| determines if the url should return a call to action image.
+  // |transparent| determines whether to request a transparent logo

[Marc Treib, 2016/05/09 16:58:21]

nit: Period at the end of the line please

[atanasova, 2016/05/10 08:32:17]

Done.

   // Note: |parse_logo_response_func| and |append_queryparams_func| must be
   // suitable for running multiple times, concurrently, and on multiple threads.
   // TODO(ianwen): remove wants_cta from parameter.
   void SetServerAPI(const GURL& logo_url,
                     const ParseLogoResponse& parse_logo_response_func,
                     const AppendQueryparamsToLogoURL& append_queryparams_func,
-                    bool wants_cta);
+                    bool wants_cta,
+                    bool transparent = false);

[Marc Treib, 2016/05/09 16:58:21]

Also here: No default values :(

[atanasova, 2016/05/10 08:32:17]

Done.

 
   // Retrieves the current search provider's logo from the local cache and/or
   // over the network, and registers |observer| to be called when the cached
@@ -216,6 +219,9 @@ class LogoTracker : public net::URLFetcherDelegate {
   // If |true| request call to action in server API.
   bool wants_cta_;
 
+  // If |true| request transparent doodle.
+  bool transparent_;
+
   // False if an asynchronous task is currently running.
   bool is_idle_;