🏡 Add a ChromeHomeNewTabPage

chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoManagerImpl.java

Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoManagerImpl.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoManagerImpl.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoManagerImpl.java
new file mode 100644
index 0000000000000000000000000000000000000000..facb26a7356781b2fde260322aeeab6f65893358
--- /dev/null
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoManagerImpl.java
@@ -0,0 +1,99 @@
+// Copyright 2017 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.
+
+package org.chromium.chrome.browser.ntp;
+
+import org.chromium.base.metrics.RecordHistogram;
+import org.chromium.chrome.browser.ntp.LogoBridge.Logo;
+import org.chromium.chrome.browser.ntp.LogoBridge.LogoObserver;
+import org.chromium.chrome.browser.ntp.LogoView.LogoManager;
+import org.chromium.chrome.browser.tab.Tab;
+import org.chromium.content_public.browser.LoadUrlParams;
+import org.chromium.ui.base.PageTransition;
+
+import jp.tomorrowkey.android.gifplayer.BaseGifImage;
+
+/**
+ * An implementation of {@link LogoManager}.
+ */
+public class LogoManagerImpl implements LogoManager {
+    // UMA enum constants. CTA means the "click-to-action" icon.
+    private static final String LOGO_SHOWN_UMA_NAME = "NewTabPage.LogoShown";
+    private static final int STATIC_LOGO_SHOWN = 0;
+    private static final int CTA_IMAGE_SHOWN = 1;
+
+    private static final String LOGO_CLICK_UMA_NAME = "NewTabPage.LogoClick";
+    private static final int STATIC_LOGO_CLICKED = 0;
+    private static final int CTA_IMAGE_CLICKED = 1;
+    private static final int ANIMATED_LOGO_CLICKED = 2;
+
+    private final Tab mTab;
+    private LogoView mLogoView;
+
+    private LogoBridge mLogoBridge;
+    private String mOnLogoClickUrl;
+    private String mAnimatedLogoUrl;
+
+    private boolean mIsDestroyed;
+
+    /**
+     * Construct a new LogoManagerImpl.
+     * @param tab The tab showing the logo view.
+     * @param logoView The view that shows the search provider logo.
+     */
+    public LogoManagerImpl(Tab tab, LogoView logoView) {
+        mTab = tab;
+        mLogoView = logoView;
+        mLogoBridge = new LogoBridge(tab.getProfile());
+    }
+
+    /**
+     * Should be called when the owning class is destroyed.
+     */
+    public void destroy() {
+        mIsDestroyed = true;
+    }
+
+    @Override
+    public void onLogoClicked(boolean isAnimatedLogoShowing) {
+        if (mIsDestroyed) return;
+
+        if (!isAnimatedLogoShowing && mAnimatedLogoUrl != null) {
+            RecordHistogram.recordSparseSlowlyHistogram(LOGO_CLICK_UMA_NAME, CTA_IMAGE_CLICKED);

[Michael van Ouwerkerk, 2017/02/28 13:36:15]

I think we should separate old-style NTP histograms from Chrome Home histograms.
If we don't, any changes will be difficult to attribute as NTP and Chrome Home
might co-exist for some time while we ramp up usage of Chrome Home. What do you
think?

[Theresa, 2017/02/28 17:02:23]

I would prefer to use the same histograms because then we can rely on our
dashboards easily visualize differences between the Chrome Home and default
experiment groups. We can set these histograms as important in the variations
associated data configuration files so that we get alerts if there are
statistically significant differences. If we use separate histograms then we
would have to do that comparison/analysis manually.

+            mLogoView.showLoadingView();
+            mLogoBridge.getAnimatedLogo(new LogoBridge.AnimatedLogoCallback() {
+                @Override
+                public void onAnimatedLogoAvailable(BaseGifImage animatedLogoImage) {
+                    if (mIsDestroyed) return;
+                    mLogoView.playAnimatedLogo(animatedLogoImage);
+                }
+            }, mAnimatedLogoUrl);
+        } else if (mOnLogoClickUrl != null) {
+            RecordHistogram.recordSparseSlowlyHistogram(LOGO_CLICK_UMA_NAME,
+                    isAnimatedLogoShowing ? ANIMATED_LOGO_CLICKED : STATIC_LOGO_CLICKED);
+            mTab.loadUrl(new LoadUrlParams(mOnLogoClickUrl, PageTransition.LINK));
+        }
+    }
+
+    @Override
+    public void getSearchProviderLogo(final LogoObserver logoObserver) {
+        if (mIsDestroyed) return;
+
+        LogoObserver wrapperCallback = new LogoObserver() {
+            @Override
+            public void onLogoAvailable(Logo logo, boolean fromCache) {
+                if (mIsDestroyed) return;
+                mOnLogoClickUrl = logo != null ? logo.onClickUrl : null;
+                mAnimatedLogoUrl = logo != null ? logo.animatedLogoUrl : null;
+                if (logo != null) {
+                    RecordHistogram.recordSparseSlowlyHistogram(LOGO_SHOWN_UMA_NAME,
+                            logo.animatedLogoUrl == null ? STATIC_LOGO_SHOWN : CTA_IMAGE_SHOWN);
+                }
+                logoObserver.onLogoAvailable(logo, fromCache);
+            }
+        };
+
+        mLogoBridge.getCurrentLogo(wrapperCallback);
+    }
+}