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

Unified Diff: content/public/android/java/src/org/chromium/content/app/ChildProcessService.java

Issue 1622743005: Introduce background Download process to android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Moving away from ChildProcessLauncher Created 4 years, 11 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/app/ChildProcessService.java
diff --git a/content/public/android/java/src/org/chromium/content/app/ChildProcessService.java b/content/public/android/java/src/org/chromium/content/app/ChildProcessService.java
index 849f9c753402e4a393219a229bacde645bc97429..8c9767fffdf5e06af498b0914ccf0cfe627f5e68 100644
--- a/content/public/android/java/src/org/chromium/content/app/ChildProcessService.java
+++ b/content/public/android/java/src/org/chromium/content/app/ChildProcessService.java
@@ -51,6 +51,7 @@ import java.util.concurrent.atomic.AtomicReference;
public class ChildProcessService extends Service {
private static final String MAIN_THREAD_NAME = "ChildProcessMain";
private static final String TAG = "ChildProcessService";
+ protected static final FileDescriptorInfo[] EMPTY_FILE_DESCRIPTOR_INFO = {};
private IChildProcessCallback mCallback;
// This is the native "Main" thread for the renderer / utility process.
@@ -289,10 +290,7 @@ public class ChildProcessService extends Service {
mFdInfos = new FileDescriptorInfo[fdInfosAsParcelable.length];
System.arraycopy(fdInfosAsParcelable, 0, mFdInfos, 0, fdInfosAsParcelable.length);
} else {
- // TODO(qinmin): On earlier androird versions, a started service running in another
- // process can get killed after Chrome is killed. To work around this issue, client
- // will never bind to the service. As a result, the file descriptors needs to be
- // passed through an intent when starting the service.
+ mFdInfos = EMPTY_FILE_DESCRIPTOR_INFO;
no sievers 2016/02/02 01:37:38 can we assert it's the download process? or at lea
qinmin 2016/02/02 20:51:41 Done. Refactored the SWITCHES_* variables in Child
}
Bundle sharedRelros = bundle.getBundle(Linker.EXTRA_LINKER_SHARED_RELROS);
if (sharedRelros != null) {

Powered by Google App Engine
This is Rietveld 408576698