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

base/android/java/src/org/chromium/base/JavaThread.java

Index: base/android/java/src/org/chromium/base/JavaThread.java
diff --git a/base/android/java/src/org/chromium/base/JavaThread.java b/base/android/java/src/org/chromium/base/JavaThread.java
new file mode 100644
index 0000000000000000000000000000000000000000..7b6592f6503d97f213252e5e5fa8461988051b20
--- /dev/null
+++ b/base/android/java/src/org/chromium/base/JavaThread.java
@@ -0,0 +1,37 @@
+// 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.
+
+package org.chromium.base;
+
+import android.os.Handler;
+import android.os.HandlerThread;
+import android.os.Looper;
+import android.os.Message;
+
+@JNINamespace("base::android")
+class JavaThread {

[benm (inactive), 2013/07/17 10:52:29]

Naming nit: should this be AndroidJavaThread? Or HandlerJavaThread?

Reason being - we're doing more than just creating a java.lang.Thread.

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

I don't like Android being part of the name, it is already in an android folder.
I'm changing it to JavaHandlerThread, speak up if you prefer something else :-)

[benm (inactive), 2013/07/17 21:07:57]

Perfect!

+    final HandlerThread mThread;
+
+    private JavaThread(String name) {
+        mThread = new HandlerThread(name);
+    }
+
+    @CalledByNative
+    private static JavaThread create(String name) {
+        return new JavaThread(name);
+    }
+
+    @CalledByNative
+    private void start(final int nativeThread, final int nativeEvent) {
+        mThread.start();
+        new Handler(mThread.getLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                nativeInitializeThread(nativeThread, nativeEvent);
+            }
+        });
+    }
+
+    private native void nativeInitializeThread(int nativeJavaThread, int nativeEvent);
+}