[App banners] Start adding unit tets

Add basic unit tests for ensuring that the banner appears,
appears after being ignored, and appears after being blocked.

BUG=457414
TEST=AppBannerManagerTest

Review URL: https://codereview.chromium.org/914403006

Cr-Commit-Position: refs/heads/master@{#316690}

chrome/android/java/src/org/chromium/chrome/browser/Tab.java

@@ -2440,6 +2440,14 @@ public class Tab implements ViewGroup.OnHierarchyChangeListener,
         nativeSetInterceptNavigationDelegate(mNativeTabAndroid, delegate);
     }
 
+    /**
+     * @return the AppBannerManager.
+     */
+    @VisibleForTesting
+    public AppBannerManager getAppBannerManagerForTesting() {
+        return mAppBannerManager;
+    }
+
     /**
      * Ensures the counter is at least as high as the specified value.  The counter should always
      * point to an unused ID (which will be handed out next time a request comes in).  Exposed so

chrome/android/java/src/org/chromium/chrome/browser/banners/AppBannerManager.java

@@ -9,6 +9,7 @@ import android.text.TextUtils;
 import org.chromium.base.ApplicationStatus;
 import org.chromium.base.CalledByNative;
 import org.chromium.base.JNINamespace;
+import org.chromium.base.VisibleForTesting;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.EmptyTabObserver;
 import org.chromium.chrome.browser.Tab;
@@ -32,6 +33,9 @@ public class AppBannerManager extends EmptyTabObserver {
     /** Retrieves information about a given package. */
     private static AppDetailsDelegate sAppDetailsDelegate;
 
+    /** Whether the banners are enabled. */
+    private static Boolean sIsEnabled;
+
     /** Pointer to the native side AppBannerManager. */
     private final long mNativePointer;
 
@@ -43,7 +47,8 @@ public class AppBannerManager extends EmptyTabObserver {
      * @return True if banners are enabled, false otherwise.
      */
     public static boolean isEnabled() {
-        return nativeIsEnabled();
+        if (sIsEnabled == null) sIsEnabled = nativeIsEnabled();
+        return sIsEnabled;
     }
 
     /**
@@ -129,6 +134,24 @@ public class AppBannerManager extends EmptyTabObserver {
         };
     }
 
+    /** Enables the app banners for testing. */
+    @VisibleForTesting
+    static void setIsEnabledForTesting() {
+        sIsEnabled = true;
+    }
+
+    /** Sets a constant (in days) that gets added to the time when the current time is requested. */
+    @VisibleForTesting
+    static void setTimeDeltaForTesting(int days) {
+        nativeSetTimeDeltaForTesting(days);
+    }
+
+    /** Records how many native BitmapFetchers are actively retrieving app icons. */
+    @VisibleForTesting
+    public int getNumActiveFetchersForTesting() {
+        return nativeGetNumActiveFetchers(mNativePointer);
+    }
+
     private static native boolean nativeIsEnabled();
     private native long nativeInit();
     private native void nativeDestroy(long nativeAppBannerManager);
@@ -137,6 +160,10 @@ public class AppBannerManager extends EmptyTabObserver {
     private native boolean nativeOnAppDetailsRetrieved(long nativeAppBannerManager, AppData data,
             String title, String packageName, String imageUrl);
 
+    // Testing methods.
+    private static native void nativeSetTimeDeltaForTesting(int days);
+    private native int nativeGetNumActiveFetchers(long nativeAppBannerManager);
+
     // UMA tracking.
     private static native void nativeRecordDismissEvent(int metric);
     private static native void nativeRecordInstallEvent(int metric);

chrome/browser/android/banners/app_banner_infobar_delegate.cc

@@ -10,6 +10,7 @@
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/worker_pool.h"
+#include "chrome/browser/android/banners/app_banner_manager.h"
 #include "chrome/browser/android/shortcut_helper.h"
 #include "chrome/browser/android/shortcut_info.h"
 #include "chrome/browser/android/tab_android.h"
@@ -114,12 +115,14 @@ void AppBannerInfoBarDelegate::InfoBarDismissed() {
     AppBannerSettingsHelper::RecordBannerEvent(
         web_contents, web_contents->GetURL(),
         native_app_package_,
-        AppBannerSettingsHelper::APP_BANNER_EVENT_DID_BLOCK, base::Time::Now());
+        AppBannerSettingsHelper::APP_BANNER_EVENT_DID_BLOCK,
+        AppBannerManager::GetCurrentTime());
   } else if (!web_app_data_.IsEmpty()) {
     AppBannerSettingsHelper::RecordBannerEvent(
         web_contents, web_contents->GetURL(),
         web_app_data_.start_url.spec(),
-        AppBannerSettingsHelper::APP_BANNER_EVENT_DID_BLOCK, base::Time::Now());
+        AppBannerSettingsHelper::APP_BANNER_EVENT_DID_BLOCK,
+        AppBannerManager::GetCurrentTime());
   }
 }
 
@@ -146,7 +149,7 @@ bool AppBannerInfoBarDelegate::Accept() {
         web_contents, web_contents->GetURL(),
         web_app_data_.start_url.spec(),
         AppBannerSettingsHelper::APP_BANNER_EVENT_DID_ADD_TO_HOMESCREEN,
-        base::Time::Now());
+        AppBannerManager::GetCurrentTime());
 
     ShortcutInfo info;
     info.UpdateFromManifest(web_app_data_);
@@ -201,7 +204,7 @@ void AppBannerInfoBarDelegate::OnInstallIntentReturned(
         web_contents->GetURL(),
         native_app_package_,
         AppBannerSettingsHelper::APP_BANNER_EVENT_DID_ADD_TO_HOMESCREEN,
-        base::Time::Now());
+        AppBannerManager::GetCurrentTime());
   }
 
   UpdateInstallState(env, obj);

chrome/browser/android/banners/app_banner_manager.cc

@@ -44,6 +44,7 @@ using base::android::ConvertUTF16ToJavaString;
 
 namespace {
 const char kBannerTag[] = "google-play-id";
+base::TimeDelta gTimeDeltaForTesting;
 }
 
 namespace banners {
@@ -144,20 +145,20 @@ void AppBannerManager::RecordCouldShowBanner(
     const std::string& package_or_start_url) {
   AppBannerSettingsHelper::RecordBannerEvent(
       web_contents(), validated_url_, package_or_start_url,
-      AppBannerSettingsHelper::APP_BANNER_EVENT_COULD_SHOW, base::Time::Now());
+      AppBannerSettingsHelper::APP_BANNER_EVENT_COULD_SHOW, GetCurrentTime());
 }
 
 bool AppBannerManager::CheckIfShouldShow(
     const std::string& package_or_start_url) {
   if (!AppBannerSettingsHelper::ShouldShowBanner(web_contents(), validated_url_,
                                                  package_or_start_url,
-                                                 base::Time::Now())) {
+                                                 GetCurrentTime())) {
     return false;
   }
 
   AppBannerSettingsHelper::RecordBannerEvent(
       web_contents(), validated_url_, package_or_start_url,
-      AppBannerSettingsHelper::APP_BANNER_EVENT_DID_SHOW, base::Time::Now());
+      AppBannerSettingsHelper::APP_BANNER_EVENT_DID_SHOW, GetCurrentTime());
   return true;
 }
 
@@ -258,6 +259,10 @@ bool AppBannerManager::OnAppDetailsRetrieved(JNIEnv* env,
   return FetchIcon(GURL(image_url));
 }
 
+int AppBannerManager::GetNumActiveFetchers(JNIEnv* env, jobject obj) {
+  return fetcher_.get() ? 1 : 0;
+}
+
 bool AppBannerManager::FetchIcon(const GURL& image_url) {
   if (!web_contents())
     return false;
@@ -284,6 +289,11 @@ int AppBannerManager::GetPreferredIconSize() {
   return Java_AppBannerManager_getPreferredIconSize(env, jobj.obj());
 }
 
+// static
+base::Time AppBannerManager::GetCurrentTime() {
+  return base::Time::Now() + gTimeDeltaForTesting;
+}
+
 void RecordDismissEvent(JNIEnv* env, jclass clazz, jint metric) {
   banners::TrackDismissEvent(metric);
 }
@@ -302,6 +312,10 @@ jboolean IsEnabled(JNIEnv* env, jclass clazz) {
       switches::kEnableAppInstallAlerts);
 }
 
+void SetTimeDeltaForTesting(JNIEnv* env, jclass clazz, jint days) {
+  gTimeDeltaForTesting = base::TimeDelta::FromDays(days);
+}
+
 // Register native methods
 bool RegisterAppBannerManager(JNIEnv* env) {
   return RegisterNativesImpl(env);

chrome/browser/android/banners/app_banner_manager.h

@@ -10,6 +10,7 @@
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
+#include "base/time/time.h"
 #include "chrome/browser/android/banners/app_banner_infobar_delegate.h"
 #include "chrome/browser/bitmap_fetcher/bitmap_fetcher.h"
 #include "chrome/browser/ui/android/infobars/app_banner_infobar.h"
@@ -93,6 +94,12 @@ class AppBannerManager : public chrome::BitmapFetcherDelegate,
   // Returns |false| if this couldn't be kicked off.
   bool FetchIcon(const GURL& image_url);
 
+  // Return how many fetchers are active.
+  int GetNumActiveFetchers(JNIEnv* env, jobject jobj);
+
+  // Returns the current time.
+  static base::Time GetCurrentTime();
+
   // WebContentsObserver overrides.
   void DidNavigateMainFrame(
       const content::LoadCommittedDetails& details,

chrome/test/data/android/banners/native_app_test.html

+<html>
+  <head>
+    <title>AppBannerManager test page</title>
+    <meta name="google-play-id" content="test.package" />
+  </head>
+  <body>
+    Promoting the "test.package" package.
+  </body>
+</html>