Chromium Code Reviews| 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) { |