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

components/search_provider_logos/google_logo_api.cc

Index: components/search_provider_logos/google_logo_api.cc
diff --git a/components/search_provider_logos/google_logo_api.cc b/components/search_provider_logos/google_logo_api.cc
index cb11bddd9856e1f705136c39e9d930b1e272ed4f..3603a6ca97948f9f480bee74406d5ccc14fa49a0 100644
--- a/components/search_provider_logos/google_logo_api.cc
+++ b/components/search_provider_logos/google_logo_api.cc
@@ -22,7 +22,8 @@ const char kResponsePreamble[] = ")]}'";
 
 GURL GoogleAppendQueryparamsToLogoURL(const GURL& logo_url,
                                       const std::string& fingerprint,
-                                      bool wants_cta) {
+                                      bool wants_cta,
+                                      bool transparent) {
   // Note: we can't just use net::AppendQueryParameter() because it escapes
   // ":" to "%3A", but the server requires the colon not to be escaped.
   // See: http://crbug.com/413845
@@ -37,12 +38,16 @@ GURL GoogleAppendQueryparamsToLogoURL(const GURL& logo_url,
     query += "async=";
     if (!fingerprint.empty()) {
       query += "es_dfp:" + fingerprint;
-      if (wants_cta)
+      if (wants_cta || transparent)

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

Hm, maybe it's time to push all the params into a vector and join them using
base::JoinString, instead of manually fiddling with commas? WDYT?

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

Done.

         query += ",";
     }
     if (wants_cta) {
       query += "cta:1";
+      if (transparent)
+        query += ",";
     }
+    if (transparent)
+      query += "transp:1";
     GURL::Replacements replacements;
     replacements.SetQueryStr(query);
     return logo_url.ReplaceComponents(replacements);