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

Unified Diff: base/android/java/src/org/chromium/base/JavaThread.java

Issue 18584006: Making a way to create thread with a Java Looper for Android (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Style fixes, and make the waitable event a stack variable Created 7 years, 5 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: 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 H
Kristian Monsen 2013/07/17 21:02:37 I don't like Android being part of the name, it is
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);
+}

Powered by Google App Engine
This is Rietveld 408576698