Allow installing apps via the AppBannerInfoBar

chrome/browser/android/banners/app_banner_manager.cc

 // 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.
 
 #include "chrome/browser/android/banners/app_banner_manager.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 81 matching lines @@
 }
 
 void AppBannerManager::OnInfoBarDestroyed() {
   weak_infobar_ptr_ = nullptr;
 }
 
 bool AppBannerManager::OnButtonClicked() const {
   if (!web_contents())
     return true;
 
-  if (!web_app_data_.IsEmpty()) {
+  if (!native_app_data_.is_null()) {
+    JNIEnv* env = base::android::AttachCurrentThread();
+    ScopedJavaLocalRef<jobject> jobj = weak_java_banner_view_manager_.get(env);
+    if (jobj.is_null())
+      return true;
+
+    return Java_AppBannerManager_installOrOpenNativeApp(env,
+                                                        jobj.obj(),
+                                                        native_app_data_.obj());
+  } else if (!web_app_data_.IsEmpty()) {
     AppBannerSettingsHelper::RecordBannerEvent(
         web_contents(), web_contents()->GetURL(),
         web_app_data_.start_url.spec(),
         AppBannerSettingsHelper::APP_BANNER_EVENT_DID_ADD_TO_HOMESCREEN,
         base::Time::Now());
 
     InstallManifestApp(web_app_data_, *app_icon_.get());
     return true;
   }
 
   return true;
 }
 
+bool AppBannerManager::OnLinkClicked() const {
+  if (!web_contents())
+    return true;
+
+  if (!native_app_data_.is_null()) {
+    // Try to show the details for the native app.
+    JNIEnv* env = base::android::AttachCurrentThread();
+    ScopedJavaLocalRef<jobject> jobj = weak_java_banner_view_manager_.get(env);
+    if (jobj.is_null())
+      return true;
+
+    Java_AppBannerManager_showAppDetails(env,
+                                         jobj.obj(),
+                                         native_app_data_.obj());
+    return true;
+  } else {
+    // Nothing should happen if the user is installing a web app.
+    return false;
+  }
+}
+
 base::string16 AppBannerManager::GetTitle() const {
   return app_title_;
 }
 
 gfx::Image AppBannerManager::GetIcon() const {
   return gfx::Image::CreateFrom1xBitmap(*app_icon_.get());
 }
 
 void AppBannerManager::ReplaceWebContents(JNIEnv* env,
                                           jobject obj,
@@ skipping 186 matching lines @@
   if (validated_url_ != web_contents()->GetURL())
     return false;
 
   std::string image_url = ConvertJavaStringToUTF8(env, jicon_url);
   app_title_ = ConvertJavaStringToUTF16(env, japp_title);
   native_app_package_ = ConvertJavaStringToUTF8(env, japp_package);
   native_app_data_.Reset(env, japp_data);
   return FetchIcon(GURL(image_url));
 }
 
+void AppBannerManager::OnIntentCompleted(JNIEnv* env,
+                                         jobject obj,
+                                         jboolean jis_installing) {
+  if (!weak_infobar_ptr_)
+    return;
+
+  if (jis_installing) {
+    AppBannerSettingsHelper::RecordBannerEvent(
+        web_contents(),
+        web_contents()->GetURL(),
+        native_app_package_,
+        AppBannerSettingsHelper::APP_BANNER_EVENT_DID_ADD_TO_HOMESCREEN,
+        base::Time::Now());
+
+    ScopedJavaLocalRef<jobject> jobj = weak_java_banner_view_manager_.get(env);

[newt (away), 2015/02/06 22:46:27]

what if jobj is null? Same issue below.

[gone, 2015/02/06 23:32:08]

Null checking now.

+    Java_AppBannerManager_monitorInstall(env,

[newt (away), 2015/02/06 22:46:27]

Why not call monitorInstall() from Java?

[gone, 2015/02/06 22:55:01]

AppData isn't available from there; it doesn't store it.

+                                         jobj.obj(),
+                                         native_app_data_.obj());
+  }
+
+  UpdateInstallState(env, obj);
+}
+
+void AppBannerManager::OnInstallFinished(JNIEnv* env,
+                                         jobject obj,
+                                         jboolean success) {
+  if (!weak_infobar_ptr_)
+    return;
+
+  if (success) {
+    UpdateInstallState(env, obj);
+  } else {
+    InfoBarService* service = InfoBarService::FromWebContents(web_contents());
+    service->RemoveInfoBar(weak_infobar_ptr_);
+  }
+}
+
+void AppBannerManager::UpdateInstallState(JNIEnv *env, jobject obj) {
+  if (!weak_infobar_ptr_ || native_app_data_.is_null())
+    return;
+
+  ScopedJavaLocalRef<jobject> jobj = weak_java_banner_view_manager_.get(env);
+  int newState = Java_AppBannerManager_determineInstallState(
+      env,
+      jobj.obj(),
+      native_app_data_.obj());
+  weak_infobar_ptr_->OnInstallStateChanged(newState);
+}
+
 bool AppBannerManager::FetchIcon(const GURL& image_url) {
   if (!web_contents())
     return false;
 
   // Begin asynchronously fetching the app icon.
   Profile* profile =
       Profile::FromBrowserContext(web_contents()->GetBrowserContext());
   fetcher_.reset(new chrome::BitmapFetcher(image_url, this));
   fetcher_.get()->Start(
       profile->GetRequestContext(),
@@ skipping 44 matching lines @@
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableAppInstallAlerts);
 }
 
 // Register native methods
 bool RegisterAppBannerManager(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace banners