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

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

Issue 2017963003: Upstream: ChildProcessLauncher connects renderer processes of WebAPKs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove PROCESS_WEBAPK_CHILD. Created 4 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/java/src/org/chromium/chrome/browser/prerender/ChromePrerenderService.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/prerender/ChromePrerenderService.java b/chrome/android/java/src/org/chromium/chrome/browser/prerender/ChromePrerenderService.java
index 6dad228692e9b60541fd17e949d2af0741a84ad7..f885bb130edde9b11738070375dd4a8440326a0f 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/prerender/ChromePrerenderService.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/prerender/ChromePrerenderService.java
@@ -39,25 +39,6 @@ public class ChromePrerenderService extends Service {
public static final String KEY_PRERENDER_HEIGHT = "prerender_height";
public static final String KEY_REFERRER = "referrer";
- private static class LauncherWarmUpTaskParams {
- final Context mContext;
- final ChildProcessCreationParams mParams;
-
- LauncherWarmUpTaskParams(Context context, ChildProcessCreationParams params) {
- mContext = context;
- mParams = params;
- }
- }
-
- private static class LauncherWarmUpTask
- extends AsyncTask<LauncherWarmUpTaskParams, Void, Void> {
- @Override
- protected Void doInBackground(LauncherWarmUpTaskParams... args) {
- ChildProcessLauncher.warmUp(args[0].mContext, args[0].mParams);
- return null;
- }
- }
-
/**
* Handler of incoming messages from clients.
*/
@@ -90,8 +71,14 @@ public class ChromePrerenderService extends Service {
try {
final Context context = getApplicationContext();
final ChromeApplication chrome = (ChromeApplication) context;
- new LauncherWarmUpTask().execute(new LauncherWarmUpTaskParams(
- context, chrome.getChildProcessCreationParams()));
+ ChildProcessCreationParams.set(chrome.getChildProcessCreationParams());
+ new AsyncTask<Context, Void, Void>() {
+ @Override
+ protected Void doInBackground(Context... params) {
+ ChildProcessLauncher.warmUp(params[0]);
+ return null;
+ }
+ }.execute(context);
ChromeBrowserInitializer.getInstance(this).handleSynchronousStartup();
ApplicationInitialization.enableFullscreenFlags(

Powered by Google App Engine
This is Rietveld 408576698