[Android:WebAPK] Don't add webapp to homescreen if WebAPK install times out part 1/3

chrome/browser/android/webapk/webapk_install_service.h

 // Copyright 2016 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.
 
 #ifndef CHROME_BROWSER_ANDROID_WEBAPK_WEBAPK_INSTALL_SERVICE_H_
 #define CHROME_BROWSER_ANDROID_WEBAPK_WEBAPK_INSTALL_SERVICE_H_
 
 #include <map>
 #include <set>
 #include <string>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
-#include "chrome/browser/android/webapk/webapk_installer.h"
 #include "components/keyed_service/core/keyed_service.h"
 #include "url/gurl.h"
 
 namespace content {
 class BrowserContext;
 }
 
 struct ShortcutInfo;
 class SkBitmap;
 
+// A Java counterpart will be generated for this enum.
+// GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.webapps
+enum class WebApkInstallResult {
+  SUCCESS = 0,
+  FAILURE = 1,
+  // An install was initiated but it timed out. We did not get a response from
+  // Google Play (or the Android OS in the case of the "unknown sources" flow)
+  // so it is possible that the install will complete some time in the future.
+  PROBABLE_FAILURE = 2
+};
+
 // Service which talks to Chrome WebAPK server and Google Play to generate a
 // WebAPK on the server, download it, and install it.
 class WebApkInstallService : public KeyedService {
  public:
-  using FinishCallback = WebApkInstaller::FinishCallback;
+  // Called when the creation/updating of a WebAPK is finished or failed.
+  // Parameters:
+  // - the result of the installation.
+  // - true if Chrome received a "request updates less frequently" directive.
+  //   from the WebAPK server.
+  // - the package name of the WebAPK.
+  using FinishCallback =
+      base::Callback<void(WebApkInstallResult, bool, const std::string&)>;
 
   static WebApkInstallService* Get(content::BrowserContext* browser_context);
 
   explicit WebApkInstallService(content::BrowserContext* browser_context);
   ~WebApkInstallService() override;
 
   // Returns whether an install for |web_manifest_url| is in progress.
   bool IsInstallInProgress(const GURL& web_manifest_url);
 
   // Talks to the Chrome WebAPK server to generate a WebAPK on the server and to
@@ skipping 12 matching lines @@
       const std::string& webapk_package,
       int webapk_version,
       const std::map<std::string, std::string>& icon_url_to_murmur2_hash,
       bool is_manifest_stale,
       const FinishCallback& finish_callback);
 
  private:
   // Called once the install/update completed or failed.
   void OnFinishedInstall(const GURL& web_manifest_url,
                          const FinishCallback& finish_callback,
-                         bool success,
+                         WebApkInstallResult result,
                          bool relax_updates,
                          const std::string& webapk_package_name);
 
   content::BrowserContext* browser_context_;
 
   // In progress installs.
   std::set<GURL> installs_;
 
   // Used to get |weak_ptr_|.
   base::WeakPtrFactory<WebApkInstallService> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebApkInstallService);
 };
 
 #endif  // CHROME_BROWSER_ANDROID_WEBAPK_WEBAPK_INSTALL_SERVICE_H_