Retry WebAPK download if fails for the first time.

chrome/browser/android/webapk/webapk_installer.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_INSTALLER_H_
 #define CHROME_BROWSER_ANDROID_WEBAPK_WEBAPK_INSTALLER_H_
 
 #include <jni.h>
 #include <memory>
 
@@ skipping 131 matching lines @@
   // WebAPK.
   void SendRequest(std::unique_ptr<webapk::WebApk> request_proto,
                    net::URLFetcher::RequestType request_type,
                    const GURL& server_url);
 
   // Called with the URL of generated WebAPK and the package name that the
   // WebAPK should be installed at.
   void OnGotWebApkDownloadUrl(const GURL& download_url,
                               const std::string& package_name);
 
+  // Downloads the WebAPK from the given |donwload_url|.

[pkotwicz, 2016/09/27 19:47:28]

Nit: |donwload_url| -> |download_url|.

[Xi Han, 2016/09/27 20:53:31]

Done.

+  void StartDownloadingWebApk(const base::FilePath& output_path,
+                              const GURL& download_url,
+                              bool retry_if_fails);
+
   // Called once the WebAPK has been downloaded. Makes the downloaded WebAPK
   // world readable and installs the WebAPK if the download was successful.
   // |file_path| is the file path that the WebAPK was downloaded to.
+  // If |retry_if_fails| is true, will sleep 2 seconds and retry the download.

[pkotwicz, 2016/09/27 19:47:28]

You should post a delayed task instead of sleeping. A friendly reminder to
update the documentation here

[Xi Han, 2016/09/27 20:53:31]

Done.

   void OnWebApkDownloaded(const base::FilePath& file_path,
+                          const GURL& download_url,
+                          bool retry_if_fails,
                           FileDownloader::Result result);
 
   // Called once the downloaded WebAPK has been made world readable. Installs
   // the WebAPK.
   // |file_path| is the file path that the WebAPK was downloaded to.
   // |change_permission_success| is whether the WebAPK could be made world
   // readable.
   void OnWebApkMadeWorldReadable(const base::FilePath& file_path,
                                  bool change_permission_success);
 
@@ skipping 57 matching lines @@
   // Points to the Java Object.
   base::android::ScopedJavaGlobalRef<jobject> java_ref_;
 
   // Used to get |weak_ptr_|.
   base::WeakPtrFactory<WebApkInstaller> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebApkInstaller);
 };
 
 #endif  // CHROME_BROWSER_ANDROID_WEBAPK_WEBAPK_INSTALLER_H_