Making a way to create thread with a Java Looper for Android

base/android/java_thread.h

+// Copyright (c) 2013 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.
+
+#ifndef BASE_THREADING_JAVA_THREAD_H_
+#define BASE_THREADING_JAVA_THREAD_H_
+
+#include <jni.h>
+
+#include "base/android/scoped_java_ref.h"
+#include "base/memory/scoped_ptr.h"
+
+namespace base {
+
+class MessageLoop;
+class WaitableEvent;
+
+namespace android {
+
+// A Java Thread with a native message loop. To run tasks, post them
+// to the message loop and they will be scheduled along Java tasks
+// on the thread.
+// This is useful for callbacks where the receiver expects a thread
+// with a prepared Looper.
+class JavaThread {
+ public:
+  JavaThread(const char* name);
+  virtual ~JavaThread() {}
+
+  base::MessageLoop* message_loop() const { return message_loop_.get(); }
+  void Start();
+  void Stop();
+
+  // Called from java on the newly created thread.
+  // Start() will not return before this methods has finished.
+  void InitializeThread(JNIEnv* env, jobject obj, jint event);

[joth, 2013/07/17 16:29:18]

guess this can be private

[Kristian Monsen, 2013/07/17 21:02:37]

Done.

[Kristian Monsen, 2013/07/17 21:54:40]

Actually it cannot be private since it is called by the generated JNI file for
convenience.

+
+  static bool RegisterBindings(JNIEnv* env);
+
+ private:
+  scoped_ptr<base::MessageLoop> message_loop_;
+  ScopedJavaGlobalRef<jobject> java_thread_;
+
+  bool started_;
+};
+
+}  // namespace android
+

[joth, 2013/07/17 16:29:18]

uber nit: no blank line here (nor in the .cc, nor in the open-namespace when
there's nothing in between them)

[Kristian Monsen, 2013/07/17 21:02:37]

Done. You mean in general for closing two namespaces?

+}  // namespace base
+
+#endif  // BASE_THREADING_JAVA_THREAD_H_