Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java

Issue 1343913002: Introduce Animated Logo to Chrome on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
index 8654f436c7b20c8ae97221a8959c0adc4a76f973..36066b468c8cd68642ec9788c805141968a5c9b2 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
@@ -54,6 +54,8 @@ import org.chromium.ui.text.SpanApplier.SpanInfo;
import java.util.Locale;
+import jp.tomorrowkey.android.gifplayer.BaseGifImage;
+
/**
* The native new tab page, represented by some basic data such as title and url, and an Android
* View that displays the page.
@@ -199,9 +201,10 @@ public class NewTabPageView extends FrameLayout
FaviconAvailabilityCallback callback);
/**
- * Navigates to a URL chosen by the search provider when the user clicks on the logo.
+ * Called when the user clicks on the logo.
+ * @param isAnimatedLogoShowing Whether the animated GIF logo is playing.
*/
- void openLogoLink();
+ void onLogoClicked(boolean isAnimatedLogoShowing);
/**
* Gets the default search provider's logo and calls logoObserver with the result.
@@ -538,6 +541,20 @@ public class NewTabPageView extends FrameLayout
}
/**
+ * Starts playing the given animated GIF logo.
+ */
+ void playAnimatedLogo(BaseGifImage gifImage) {
+ mSearchProviderLogoView.playAnimatedLogo(gifImage);
+ }
+
+ /**
+ * @return Whether the GIF animation is playing in the logo.
+ */
+ boolean isAnimatedLogoShowing() {
+ return mSearchProviderLogoView.isAnimatedLogoShowing();
+ }
+
+ /**
* @return Whether URL focus animations are currently disabled.
*/
boolean urlFocusAnimationsDisabled() {
@@ -700,7 +717,7 @@ public class NewTabPageView extends FrameLayout
* InvalidationAwareThumbnailProvider#captureThumbnail(Canvas)
*/
void captureThumbnail(Canvas canvas) {
- mSearchProviderLogoView.endAnimation();
+ mSearchProviderLogoView.endFadeAnimation();
ViewUtils.captureBitmap(this, canvas);
mSnapshotWidth = getWidth();
mSnapshotHeight = getHeight();
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java ('k') | chrome/browser/android/logo_bridge.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698