Make AppBannerInfoBar install WebAPK.

chrome/browser/android/webapk/webapk_installer_unittest.cc

Index: chrome/browser/android/webapk/webapk_installer_unittest.cc
diff --git a/chrome/browser/android/webapk/webapk_installer_unittest.cc b/chrome/browser/android/webapk/webapk_installer_unittest.cc
index 0231684ca8845d6a97fb13a4a08752c5073b55ca..f003add968be5c53df9be45e30ca9d07577563bf 100644
--- a/chrome/browser/android/webapk/webapk_installer_unittest.cc
+++ b/chrome/browser/android/webapk/webapk_installer_unittest.cc
@@ -66,6 +66,7 @@ class TestWebApkInstaller : public WebApkInstaller {
       JNIEnv* env,
       const base::android::ScopedJavaLocalRef<jstring>& file_path,
       const base::android::ScopedJavaLocalRef<jstring>& package_name) override {
+    OnSuccess();

[gone, 2016/08/29 17:24:09]

This isn't exactly like the code flow it's testing because this calls
OnSuccess() immediately, while the other one is asynchronous.  Can you post a
task to run after it returns from this function?

[Xi Han, 2016/08/29 17:55:46]

Done.

     return true;
   }
 
@@ -73,6 +74,7 @@ class TestWebApkInstaller : public WebApkInstaller {
       JNIEnv* env,
       const base::android::ScopedJavaLocalRef<jstring>& file_path,
       const base::android::ScopedJavaLocalRef<jstring>& package_name) override {
+    OnSuccess();
     return true;
   }
 
@@ -134,7 +136,7 @@ class WebApkInstallerRunner {
   bool success() { return success_; }
 
  private:
-  void OnCompleted(bool success) {
+  void OnCompleted(bool success, const std::string& webapk_package) {
     success_ = success;
     on_completed_callback_.Run();
   }