Chromium Code Reviews| 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..4432c5c06b31c975cef71877bbb4c6992530d913 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 |
| @@ -5,6 +5,7 @@ |
| package org.chromium.chrome.browser.webapps; |
| import android.content.Intent; |
| +import android.os.AsyncTask; |
| import org.chromium.base.ContextUtils; |
| import org.chromium.base.library_loader.LibraryProcessType; |
| @@ -149,12 +150,17 @@ public class WebApkActivity extends WebappActivity { |
| public void finishNativeInitialization() { |
| super.finishNativeInitialization(); |
| - // TODO(hanxi): Ask WebApk's update manager whether to check resource updates. |
| - if (mManifestUpgradeDetector == null) { |
| - mManifestUpgradeDetector = |
| - new ManifestUpgradeDetector(getActivityTab(), mWebappInfo); |
| - } |
| - mManifestUpgradeDetector.start(); |
| + new AsyncTask<Void, Void, Void>() { |
| + @Override |
| + protected Void doInBackground(Void... nothing) { |
| + if (mManifestUpgradeDetector == null) { |
| + mManifestUpgradeDetector = |
| + new ManifestUpgradeDetector(getActivityTab(), mWebappInfo); |
| + } |
| + WebApkUpdateManager.checkUpdate(mWebappInfo.id(), mManifestUpgradeDetector); |
|
pkotwicz
2016/08/08 18:59:15
I just talked to Yaron. He said that it is OK for
Xi Han
2016/08/08 21:25:05
As a second thought, move the call of |callUpdate|
|
| + return null; |
| + } |
| + }.execute(); |
| } |
| @Override |