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

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

Issue 2933783002: [Android WebAPK] Change WebAPK update into two phases (Closed)
Patch Set: Merge branch 'master' into background_updates0 Created 3 years, 6 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/junit/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java b/chrome/android/junit/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
index 3dbb28fc41d063e42b2b3540e3dbdb6910976db7..b3717f52e9daf6d566ad59eab6621f4be8eb1de1 100644
--- a/chrome/android/junit/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
@@ -144,15 +144,14 @@ public class WebApkUpdateManagerTest {
}
@Override
- protected void scheduleUpdate(WebApkInfo info, String primaryIconUrl, String badgeIconUrl,
- boolean isManifestStale) {
+ protected void buildProtoAndScheduleUpdate(WebApkInfo info, String primaryIconUrl,
+ String badgeIconUrl, boolean isManifestStale) {
mUpdateName = info.name();
- super.scheduleUpdate(info, primaryIconUrl, badgeIconUrl, isManifestStale);
+ scheduleUpdate(info, new byte[0]);
}
@Override
- protected void updateAsyncImpl(WebApkInfo info, String primaryIconUrl, String badgeIconUrl,
- boolean isManifestStale) {
+ protected void updateAsyncImpl(WebApkInfo info, byte[] serializedProto) {
mUpdateRequested = true;
}

Powered by Google App Engine
This is Rietveld 408576698