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..f908bf185415612845a839f087cf9a0bf1a17f18 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
@@ -35,14 +36,16 @@ GURL GoogleAppendQueryparamsToLogoURL(const GURL& logo_url,
       query += "&";
 
     query += "async=";
-    if (!fingerprint.empty()) {
-      query += "es_dfp:" + fingerprint;
-      if (wants_cta)
-        query += ",";
-    }
-    if (wants_cta) {
-      query += "cta:1";
-    }
+    std::vector<std::string> params;
+    if (!fingerprint.empty())
+      params.push_back("es_dfp:" + fingerprint);
+
+    if (wants_cta)
+      params.push_back("cta:1");
+
+    if (transparent)
+      params.push_back("transp:1");

[Bernhard Bauer, 2016/05/10 09:15:24]

Nit: empty line afterwards.

[atanasova, 2016/05/10 13:51:25]

Done.

+    query += base::JoinString(params, ",");
     GURL::Replacements replacements;
     replacements.SetQueryStr(query);
     return logo_url.ReplaceComponents(replacements);