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

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

Issue 2515293004: Chrome talks to Play to install WebAPKs. (Closed)
Patch Set: Don't use play install in webapk_installer_unittest. Created 4 years 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/DeferredStartupHandler.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java b/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
index 7ad77580f366fc2086989e3f2977bc33ccb72788..78d51044f76f1021d336c4afe5d437bcad5fef24 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
@@ -46,6 +46,7 @@ import org.chromium.chrome.browser.preferences.ChromePreferenceManager;
import org.chromium.chrome.browser.preferences.privacy.PrivacyPreferencesManager;
import org.chromium.chrome.browser.share.ShareHelper;
import org.chromium.chrome.browser.webapps.ChromeWebApkHost;
+import org.chromium.chrome.browser.webapps.WebApkInstaller;
import org.chromium.chrome.browser.webapps.WebApkVersionManager;
import org.chromium.chrome.browser.webapps.WebappRegistry;
import org.chromium.components.minidump_uploader.CrashFileManager;
@@ -271,6 +272,12 @@ public class DeferredStartupHandler {
if (ChromeWebApkHost.isEnabled()) {
WebApkVersionManager.updateWebApksIfNeeded();
+ if (ChromeWebApkHost.canUseGooglePlayToInstallWebApk()) {
pkotwicz 2016/12/02 22:27:04 Shouldn't this be called prior to WebApkVersionMan
Xi Han 2016/12/05 17:27:54 Hmm, I don't think WebApkVersionManager#updateWebA
+ ChromeApplication application =
+ (ChromeApplication) ContextUtils.getApplicationContext();
pkotwicz 2016/12/02 22:27:04 Shoudn't you use mAppContext?
Xi Han 2016/12/05 17:27:54 Done.
+ WebApkInstaller.setGooglePlayWebApkInstallDelegate(
+ application.createGooglePlayWebApkInstallDelegate());
+ }
}
removeSnapshotDatabase();

Powered by Google App Engine
This is Rietveld 408576698