Add a feature flag for enabling transparent doodle request.

chrome/browser/android/logo_service.cc

Index: chrome/browser/android/logo_service.cc
diff --git a/chrome/browser/android/logo_service.cc b/chrome/browser/android/logo_service.cc
index ff639514b5eaaccd4bbcef088bc59d9bbf192e91..50556b24b73e039091d56869351f74e813886ddd 100644
--- a/chrome/browser/android/logo_service.cc
+++ b/chrome/browser/android/logo_service.cc
@@ -4,9 +4,11 @@
 
 #include "chrome/browser/android/logo_service.h"
 
+#include "base/feature_list.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/thread_task_runner_handle.h"
+#include "chrome/browser/android/chrome_feature_list.h"
 #include "chrome/browser/image_decoder.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
@@ -125,7 +127,8 @@ void LogoService::GetLogo(search_provider_logos::LogoObserver* observer) {
       GetGoogleDoodleURL(profile_),
       base::Bind(&search_provider_logos::GoogleParseLogoResponse),
       base::Bind(&search_provider_logos::GoogleAppendQueryparamsToLogoURL),
-      true);
+      true,

[tschumann, 2016/05/10 08:44:34]

a comment explaining the semantics of the bools would be nice. Not sure what the
chrome way of doing this is; something like:
/*wants_cta=*/true,
or
true /* wants_cta */

would be nice.

[Marc Treib, 2016/05/10 08:54:59]

Haha, I just made the exact same comment on the other CL :D
I think the second way is more common in Chromiuem, but I've seen both. Either's
fine IMO.

[atanasova, 2016/05/10 09:12:54]

Done.

[tschumann, 2016/05/10 09:19:22]

To follow up: In google3 there's a recent trend to /*foo=*/bar, as we have a
supporting critique finding which will tell you if the comment is wrong -- i.e.
if the argument name is different.

+      base::FeatureList::IsEnabled(chrome::android::kNTPSnippetsFeature));
   logo_tracker_->GetLogo(observer);
 }