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

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

Issue 2948293002: [Fork M59] android: Warmup after library load (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/init/AsyncInitTaskRunner.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/init/AsyncInitTaskRunner.java b/chrome/android/java/src/org/chromium/chrome/browser/init/AsyncInitTaskRunner.java
index 73a6dd32d0eb6094a6e42f5f61778c0061c990ae..b87631a73d31fd448425e7966341dc4614aa8cbe 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/init/AsyncInitTaskRunner.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/init/AsyncInitTaskRunner.java
@@ -38,9 +38,9 @@ public abstract class AsyncInitTaskRunner {
return ChromeVersionInfo.isOfficialBuild();
}
- private class LoadTask extends AsyncTask<Void, Void, Boolean> {
+ private class LoadTask extends AsyncTask<Boolean, Void, Boolean> {
@Override
- protected Boolean doInBackground(Void... params) {
+ protected Boolean doInBackground(Boolean... allocateChildConnection) {
try {
LibraryLoader libraryLoader = LibraryLoader.get(LibraryProcessType.PROCESS_BROWSER);
libraryLoader.ensureInitialized();
@@ -58,6 +58,9 @@ public abstract class AsyncInitTaskRunner {
} catch (ProcessInitException e) {
return false;
}
+ if (allocateChildConnection[0]) {
+ ChildProcessLauncher.warmUp(ContextUtils.getApplicationContext());
+ }
return true;
}
@@ -111,11 +114,8 @@ public abstract class AsyncInitTaskRunner {
});
}
- if (allocateChildConnection) {
- ChildProcessLauncher.warmUp(ContextUtils.getApplicationContext());
- }
mLoadTask = new LoadTask();
- mLoadTask.executeOnExecutor(getExecutor());
+ mLoadTask.executeOnExecutor(getExecutor(), allocateChildConnection);
}
private void tasksPossiblyComplete(boolean result) {
« no previous file with comments | « no previous file | content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698