Make AppBannerInfoBar install WebAPK.

chrome/browser/android/webapk/webapk_installer.h

Index: chrome/browser/android/webapk/webapk_installer.h
diff --git a/chrome/browser/android/webapk/webapk_installer.h b/chrome/browser/android/webapk/webapk_installer.h
index 8c1f3d8855442b8110d273a4c2c13f5695087511..df30299224fa485f19f05418a44124cd1d4998f3 100644
--- a/chrome/browser/android/webapk/webapk_installer.h
+++ b/chrome/browser/android/webapk/webapk_installer.h
@@ -13,6 +13,7 @@
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/timer/timer.h"
+#include "chrome/browser/android/shortcut_helper.h"
 #include "chrome/browser/android/shortcut_info.h"
 #include "chrome/browser/net/file_downloader.h"
 #include "net/url_request/url_fetcher.h"
@@ -51,13 +52,16 @@ class WebApkInstaller : public net::URLFetcherDelegate {
   // Talks to the Chrome WebAPK server to generate a WebAPK on the server and to
   // Google Play to install the downloaded WebAPK. Calls |callback| after the
   // request to install the WebAPK is sent to Google Play.
-  void InstallAsync(content::BrowserContext* browser_context,
-                    const FinishCallback& callback);
+  void InstallAsync(
+      content::BrowserContext* browser_context,
+      const ShortcutHelper::WebApkPackageNameAvailableCallback& callback,
+      const FinishCallback& finish_callback);

[pkotwicz, 2016/08/19 22:17:37]

Can we change the callback to be called when the installation has failed or
succeeded (as opposed to when we send the install intent to Google Play)?
Can we move the WebAPK specific logic in AppBannerInfoBarDelegateAndroid.java to
WebApkInstaller.java (ApplicationStateListener and InstallerDelegate.Observer).
In my opinion it would be awesome if AppBannerInfoBarDelegateAndroid.java had no
WebAPK related code at all.

I think that my suggestion makes things simpler right now and gives us less work
once WebApkInstaller.java talks to Google Play in the future

Yaron, thoughts?

 
   // Same as InstallAsync() but uses the passed in |request_context_getter|.
   void InstallAsyncWithURLRequestContextGetter(
       net::URLRequestContextGetter* request_context_getter,
-      const FinishCallback& callback);
+      const ShortcutHelper::WebApkPackageNameAvailableCallback& callback,
+      const FinishCallback& finish_callback);
 
   // Talks to the Chrome WebAPK server to update a WebAPK on the server and to
   // the Google Play server to install the downloaded WebAPK. Calls |callback|
@@ -183,6 +187,11 @@ class WebApkInstaller : public net::URLFetcherDelegate {
   // Callback to call once WebApkInstaller succeeds or fails.
   FinishCallback finish_callback_;
 
+  // Callback to call once the WebAPK's package name is available (i.e.,
+  // received from the WebAPK Server).
+  ShortcutHelper::WebApkPackageNameAvailableCallback
+      package_available_callback_;
+
   // Web Manifest info.
   const ShortcutInfo shortcut_info_;