Introduce Animated Logo to Chrome on Android

chrome/browser/android/logo_bridge.h

 // 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.
 
 #ifndef CHROME_BROWSER_ANDROID_LOGO_BRIDGE_H_
 #define CHROME_BROWSER_ANDROID_LOGO_BRIDGE_H_
 
 #include <jni.h>
 
+#include "base/android/scoped_java_ref.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "net/url_request/url_fetcher_delegate.h"
 
 class LogoService;
 
+namespace net {
+class URLFetcher;
+class URLRequestContextGetter;
+}
+
 // The C++ counterpart to LogoBridge.java. Enables Java code to access the
 // default search provider's logo.
-class LogoBridge {
+class LogoBridge : public net::URLFetcherDelegate {
  public:
   explicit LogoBridge(jobject j_profile);
   void Destroy(JNIEnv* env, jobject obj);
   void GetCurrentLogo(JNIEnv* env, jobject obj, jobject j_logo_observer);
 
+  jboolean GetAnimatedLogo(JNIEnv* env,
+                       jobject obj,

[newt (away), 2015/09/23 20:38:44]

indentation

[Ian Wen, 2015/09/23 21:59:52]

Done.

+                       jobject j_callback,
+                       jstring j_url);
+
+  // net::URLFetcherDelegate:
+  void OnURLFetchComplete(const net::URLFetcher* source) override;
+
  private:
-  ~LogoBridge();
+  ~LogoBridge() override;
+
+  // Clears and resets the URLFetcher for animated logo.
+  void ClearFetcher();
 
   LogoService* logo_service_;
+
+  // The URLFetcher currently fetching the logo. NULL when not fetching.

[newt (away), 2015/09/23 20:38:44]

s/logo/animated logo/

[Ian Wen, 2015/09/23 21:59:52]

Done.

+  scoped_ptr<net::URLFetcher> fetcher_;
+
+  // The URLRequestContextGetter used for network requests.

[newt (away), 2015/09/23 20:38:43]

"... used to download the animated logo."

[Ian Wen, 2015/09/23 21:59:52]

Done.

+  scoped_refptr<net::URLRequestContextGetter> request_context_getter_;
+
+  base::android::ScopedJavaGlobalRef<jobject> j_callback_;
+
   base::WeakPtrFactory<LogoBridge> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(LogoBridge);
 };
 
 bool RegisterLogoBridge(JNIEnv* env);
 
 #endif  // CHROME_BROWSER_ANDROID_LOGO_BRIDGE_H_