Execute operations of ChildProcessLauncher on launcher thread

content/browser/android/child_process_launcher_android.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/android/child_process_launcher_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 212 matching lines @@
   return gfx::ScopedJavaSurface::AcquireExternalSurface(
       Java_ChildProcessLauncher_getSurfaceTextureSurface(
           env, surface_texture_id, client_id).obj());
 }
 
 jboolean IsSingleProcess(JNIEnv* env, const JavaParamRef<jclass>& clazz) {
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kSingleProcess);
 }
 
+static void ExecuteOnLauncherThread(
+    const base::android::JavaRef<jobject>& runnable) {
+  JNIEnv* env = AttachCurrentThread();
+  DCHECK(env);
+  Java_ChildProcessLauncher_executeOnLauncherThread(env, runnable.obj());
+}
+
+jboolean RunOnLauncherThread(JNIEnv* env,
+                             const JavaParamRef<jclass>& clazz,
+                             const JavaParamRef<jobject>& jrunnable,
+                             jlong delayInMS) {
+  if (jrunnable.is_null())

[Yaron, 2015/09/09 01:00:20]

Does this actually happen?

[Jaekyun Seok (inactive), 2015/09/09 02:02:41]

No. I will remove this.

+    return false;
+
+  if (!BrowserThread::IsThreadInitialized(BrowserThread::PROCESS_LAUNCHER))

[Yaron, 2015/09/09 01:00:20]

Is this also because of tests? Or can this actually happen too early? The reason
I ask is because of the complexity of the |asserted| flag you added on the java
side. I'd prefer if we can get rid of that and assume that this is always
successful

[Jaekyun Seok (inactive), 2015/09/09 02:02:41]

This happens actually when ChildProcessLauncher.warmUp() is called very early.

+    return false;
+
+  if (delayInMS > 0) {
+    ScopedJavaGlobalRef<jobject> scoped_runnable;
+    scoped_runnable.Reset(env, jrunnable.obj());
+    BrowserThread::PostDelayedTask(
+        BrowserThread::PROCESS_LAUNCHER, FROM_HERE,
+        base::Bind(&ExecuteOnLauncherThread, scoped_runnable),
+        base::TimeDelta::FromMilliseconds(delayInMS));
+  } else if (BrowserThread::CurrentlyOn(BrowserThread::PROCESS_LAUNCHER)) {
+    ExecuteOnLauncherThread(jrunnable);
+  } else {
+    ScopedJavaGlobalRef<jobject> scoped_runnable;
+    scoped_runnable.Reset(env, jrunnable.obj());
+    BrowserThread::PostTask(
+        BrowserThread::PROCESS_LAUNCHER, FROM_HERE,
+        base::Bind(&ExecuteOnLauncherThread, scoped_runnable));
+  }
+  return true;
+}
+
 bool RegisterChildProcessLauncher(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content