Introduce Animated Logo to Chrome on Android

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 5277f7efbf8cb2c0658b253666119bff79096b8d..94246dd1dea13f9ccc60310c3d6b4be2f0e04c3e 100644
--- a/chrome/browser/android/logo_service.cc
+++ b/chrome/browser/android/logo_service.cc
@@ -17,6 +17,7 @@
 #include "net/url_request/url_request_context_getter.h"
 
 using content::BrowserThread;
+using search_provider_logos::AnimatedLogoTracker;
 using search_provider_logos::Logo;
 using search_provider_logos::LogoDelegate;
 using search_provider_logos::LogoTracker;
@@ -125,10 +126,21 @@ void LogoService::GetLogo(search_provider_logos::LogoObserver* observer) {
       GetGoogleDoodleURL(profile_),
       base::Bind(&search_provider_logos::GoogleParseLogoResponse),
       base::Bind(&search_provider_logos::GoogleAppendQueryparamsToLogoURL),
-      false);
+      true);

[newt (away), 2015/09/18 20:46:03]

as a later step, we can remove this parameter from SetServerAPI since all
platforms now support animated logos :)

[Ian Wen, 2015/09/22 21:39:06]

I added a todo to clean this up.

   logo_tracker_->GetLogo(observer);
 }
 
+void LogoService::GetAnimatedLogo(
+    const GURL& animated_logo_url,
+    const search_provider_logos::AnimatedLogoCallback& callback) {
+
+  if (!animated_logo_tracker_) {
+    animated_logo_tracker_.reset(
+        new AnimatedLogoTracker(profile_->GetRequestContext()));
+  }
+  animated_logo_tracker_->GetAnimatedLogo(animated_logo_url, callback);
+}
+
 // LogoServiceFactory ---------------------------------------------------------
 
 // static