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

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

Issue 2430773002: Fix WebappDataStorage#updateDidLastWebApkUpdateRequestSucceed() corner cases (Closed)
Patch Set: Merge branch 'master' into update_fail00 Created 4 years, 2 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 d3cd11658016d254f20da4ed62d308eca87a8c78..4e438cd4d26c2c78985d5b73f41a9aff076f8f48 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
@@ -56,9 +56,10 @@ public class WebApkActivity extends WebappActivity {
@Override
public void onWebappDataStorageRetrieved(WebappDataStorage storage) {
updateStorage(storage);
- // Initialize the update related timestamps to the registration time.
+ // Initialize the time of the last is-update-needed check with the
+ // registration time. This prevents checking for updates on the first run.
storage.updateTimeOfLastCheckForUpdatedWebManifest();
- storage.updateTimeOfLastWebApkUpdateRequestCompletion();
+
// The downloading of the splash screen image happens before a WebAPK's
// package name is available. If we want to use the image in the first
// launch, we need to cache the image, register the WebAPK and store the
« no previous file with comments | « chrome/android/BUILD.gn ('k') | chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkUpdateManager.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698