Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3906)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java

Issue 2184913005: Add calls to the server to request WebAPK updates. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and nits. Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..c90ff1e660c72b2e919776b37158f9388de77088 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.updateIfNeeded(getActivityTab(), mWebappInfo);
}
@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();
}

Powered by Google App Engine
This is Rietveld 408576698