Introduce Animated Logo to Chrome on Android

chrome/browser/android/logo_service.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/logo_service.h"
 
 #include "base/memory/weak_ptr.h"
 #include "base/thread_task_runner_handle.h"
 #include "chrome/browser/image_decoder.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
 #include "chrome/browser/search_engines/ui_thread_search_terms_data.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "components/search_engines/template_url_service.h"
 #include "components/search_provider_logos/google_logo_api.h"
 #include "content/public/browser/browser_thread.h"
 #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;
 
 namespace {
 
 const char kCachedLogoDirectory[] = "Search Logo";
 const int kDecodeLogoTimeoutSeconds = 30;
 
 // Returns the URL where the doodle can be downloaded, e.g.
@@ skipping 88 matching lines @@
         BrowserThread::GetBlockingPool(),
         profile_->GetRequestContext(),
         scoped_ptr<search_provider_logos::LogoDelegate>(
             new ChromeLogoDelegate())));
   }
 
   logo_tracker_->SetServerAPI(
       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
 LogoService* LogoServiceFactory::GetForProfile(Profile* profile) {
   return static_cast<LogoService*>(
       GetInstance()->GetServiceForBrowserContext(profile, true));
 }
 
 // static
 LogoServiceFactory* LogoServiceFactory::GetInstance() {
   return base::Singleton<LogoServiceFactory>::get();
 }
 
 LogoServiceFactory::LogoServiceFactory()
     : BrowserContextKeyedServiceFactory(
           "LogoService",
           BrowserContextDependencyManager::GetInstance()) {
 }
 
 LogoServiceFactory::~LogoServiceFactory() {}
 
 KeyedService* LogoServiceFactory::BuildServiceInstanceFor(
     content::BrowserContext* context) const {
   Profile* profile = static_cast<Profile*>(context);
   DCHECK(!profile->IsOffTheRecord());
   return new LogoService(profile);
 }