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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java

Issue 2774363003: android: Java-based launcher thread (Closed)
Patch Set: gab review 2 Created 3 years, 9 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: content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java b/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java
index 162d5078d294779002786fd9c81ae9c7e50db688..be12dda26307d8e80e08510bd25e1aca17115873 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java
@@ -8,7 +8,6 @@ import android.content.ComponentName;
import android.content.Context;
import android.content.pm.ApplicationInfo;
import android.content.pm.PackageManager;
-import android.os.AsyncTask;
import android.os.Bundle;
import android.os.IBinder;
import android.os.RemoteException;
@@ -449,7 +448,7 @@ public class ChildProcessLauncher {
public void run() {
final SpawnData pendingSpawn = freeConnectionAndDequeuePending(conn);
if (pendingSpawn != null) {
- new Thread(new Runnable() {
+ LauncherThread.post(new Runnable() {
@Override
public void run() {
startInternal(pendingSpawn.context(), pendingSpawn.commandLine(),
@@ -457,7 +456,7 @@ public class ChildProcessLauncher {
pendingSpawn.launchCallback(), pendingSpawn.callbackType(),
pendingSpawn.inSandbox(), pendingSpawn.getCreationParams());
}
- }).start();
+ });
}
}
}, FREE_CONNECTION_DELAY_MILLIS);
@@ -685,7 +684,7 @@ public class ChildProcessLauncher {
@Override
public void onChildStartFailed() {
Log.e(TAG, "ChildProcessConnection.start failed, trying again");
- AsyncTask.THREAD_POOL_EXECUTOR.execute(new Runnable() {
+ LauncherThread.post(new Runnable() {
@Override
public void run() {
// The child process may already be bound to another client
« no previous file with comments | « content/public/android/BUILD.gn ('k') | content/public/android/java/src/org/chromium/content/browser/LauncherThread.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698