Fall back to shortcut A2HS when Phonesky is out of date or unavailable.

chrome/browser/android/webapk/webapk_installer.cc

Index: chrome/browser/android/webapk/webapk_installer.cc
diff --git a/chrome/browser/android/webapk/webapk_installer.cc b/chrome/browser/android/webapk/webapk_installer.cc
index 161a3881865f2dd5aa258762a891f35d4ba10ac9..e92ebb7c1d6329ad857bf0beb7e5a74a9ab80913 100644
--- a/chrome/browser/android/webapk/webapk_installer.cc
+++ b/chrome/browser/android/webapk/webapk_installer.cc
@@ -427,10 +427,8 @@ void WebApkInstaller::OnURLFetchComplete(const net::URLFetcher* source) {
   if (CanUseGooglePlayInstallService()) {
     int version = 1;
     base::StringToInt(response->version(), &version);
-    if (!InstallOrUpdateWebApkFromGooglePlay(
-        response->package_name(), version, response->token())) {
-      OnFailure();
-    }
+    InstallOrUpdateWebApkFromGooglePlay(
+        response->package_name(), version, response->token());

[pkotwicz, 2017/02/03 18:56:32]

Why this change? If we are ignoring the return value of
InstallOrUpdateWebApkFromGooglePlay(), InstallOrUpdateWebApkFromGooglePlay()
should not have a return value.

[Xi Han, 2017/02/03 22:16:21]

This is because ChromeWebApkHost::CanUseGooglePlayInstallApi(callback) always
wait for the callback to be called, even for the failure case. However, that API
eventually share the same code with InstallOrUpdateWebApkFromGooglePlay(). I
will introduce a follow up CL to remove all the return values of the
InstallOrUpdateWebApkFromGooglePlay().

[pkotwicz, 2017/02/03 23:05:05]

Please file the bug too :)

     return;
   }