Allow app banners to be triggered by increases in site engagement.

chrome/browser/android/banners/app_banner_manager_android.cc

 // Copyright 2015 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/banners/app_banner_manager_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "chrome/browser/android/banners/app_banner_data_fetcher_android.h"
 #include "chrome/browser/android/shortcut_helper.h"
@@ skipping 15 matching lines @@
 const char kIdName[] = "id";
 const char kPlayInlineReferrer[] = "playinline=chrome_inline";
 
 }  // anonymous namespace
 
 namespace banners {
 
 AppBannerManagerAndroid::AppBannerManagerAndroid(
     JNIEnv* env,
     jobject obj)
-    : AppBannerManager(),
+    : AppBannerManager(nullptr),
       weak_java_banner_view_manager_(env, obj) {
 }
 
 AppBannerManagerAndroid::~AppBannerManagerAndroid() {
 }
 
 void AppBannerManagerAndroid::Destroy(JNIEnv* env,
                                       const JavaParamRef<jobject>& obj) {
   delete this;
 }
@@ skipping 131 matching lines @@
 
 bool AppBannerManagerAndroid::IsFetcherActive(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj) {
   return AppBannerManager::IsFetcherActive();
 }
 
 void AppBannerManagerAndroid::RequestAppBanner(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj) {
-  AppBannerManager::RequestAppBanner(web_contents()->GetMainFrame(),
-                                     web_contents()->GetLastCommittedURL(),
-                                     true);
+  // Set debug mode to true as this method is only called from DevTools.
+  AppBannerManager::RequestAppBanner(web_contents()->GetLastCommittedURL(),
+                                     true /* is_debug_mode */);
 }
 
 // static
 bool AppBannerManagerAndroid::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 jlong Init(JNIEnv* env,
            const JavaParamRef<jobject>& obj) {
   AppBannerManagerAndroid* manager = new AppBannerManagerAndroid(env, obj);
@@ skipping 13 matching lines @@
 
 void SetEngagementWeights(JNIEnv* env,
                           const JavaParamRef<jclass>& clazz,
                           jdouble direct_engagement,
                           jdouble indirect_engagement) {
   AppBannerManager::SetEngagementWeights(direct_engagement,
                                          indirect_engagement);
 }
 
 }  // namespace banners