Add calls to the server to request WebAPK updates.

chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java

Index: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java
index 7c4a070defd51917c2f13376bbf5e96ed1f2895e..7016532e0ffe2f4010eae8e0237fe83bd16bc5f2 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java
@@ -27,9 +27,8 @@ import org.chromium.webapk.lib.client.WebApkServiceConnectionManager;
  * UI-less Chrome.
  */
 public class WebApkActivity extends WebappActivity {
-    /** Detects whether the associated Web Manifest changes. */
-    private ManifestUpgradeDetector mManifestUpgradeDetector = null;
-
+    /** Manages whether to check update for the WebAPK, and starts update check if needed. */
+    private WebApkUpdateManager mUpdateManager;
     @Override
     protected void onNewIntent(Intent intent) {
         super.onNewIntent(intent);
@@ -146,15 +145,11 @@ public class WebApkActivity extends WebappActivity {
     }
 
     @Override
-    public void finishNativeInitialization() {
-        super.finishNativeInitialization();
+    public void onDeferredStartup() {
+        super.onDeferredStartup();
 
-        // TODO(hanxi): Ask WebApk's update manager whether to check resource updates.
-        if (mManifestUpgradeDetector == null) {
-            mManifestUpgradeDetector =
-                    new ManifestUpgradeDetector(getActivityTab(), mWebappInfo);
-        }
-        mManifestUpgradeDetector.start();
+        mUpdateManager = new WebApkUpdateManager();
+        mUpdateManager.checkIfUpdateNeeded(getActivityTab(), mWebappInfo);

[pkotwicz, 2016/08/11 22:02:43]

Nit: The method name is deceiving. This method not only checks whether an update
is needed but also does the update if it is needed.

Perhaps rename the method to updateIfNeeded() ?

[Xi Han, 2016/08/15 21:38:44]

Done.

     }
 
     @Override
@@ -182,8 +177,8 @@ public class WebApkActivity extends WebappActivity {
 
     @Override
     protected void onDestroyInternal() {
-        if (mManifestUpgradeDetector != null) {
-            mManifestUpgradeDetector.destroy();
+        if (mUpdateManager != null) {
+            mUpdateManager.destroy();
         }
         super.onDestroyInternal();
     }