Set Realtime Audio Thread priority on Android

base/threading/platform_thread_posix.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 "base/threading/platform_thread.h"
 
 #include <errno.h>
 #include <sched.h>
 
 #include "base/lazy_instance.h"
@@ skipping 12 matching lines @@
 #if defined(OS_LINUX)
 #include <sys/prctl.h>
 #include <sys/resource.h>
 #include <sys/syscall.h>
 #include <sys/time.h>
 #include <unistd.h>
 #endif
 
 #if defined(OS_ANDROID)
 #include "base/android/jni_android.h"
+#include "jni/ThreadUtils_jni.h"
 #endif
 
 // TODO(bbudge) Use time.h when NaCl toolchain supports _POSIX_TIMERS
 #if defined(OS_NACL)
 #include <sys/nacl_syscalls.h>
 #endif
 
 namespace base {
 
 #if defined(OS_MACOSX)
@@ skipping 224 matching lines @@
 
 // static
 void PlatformThread::Join(PlatformThreadHandle thread_handle) {
   // Joining another thread may block the current thread for a long time, since
   // the thread referred to by |thread_handle| may still be running long-lived /
   // blocking tasks.
   base::ThreadRestrictions::AssertIOAllowed();
   pthread_join(thread_handle, NULL);
 }
 
-#if !defined(OS_MACOSX)
-// Mac OS X uses lower-level mach APIs.
-
+#if !defined(OS_MACOSX) && !defined(OS_ANDROID)
+// Mac OS X uses lower-level mach APIs and Android uses Java APIs.
 // static
 void PlatformThread::SetThreadPriority(PlatformThreadHandle, ThreadPriority) {
   // TODO(crogers): Implement, see http://crbug.com/116172
 }
 #endif
 
+#if defined(OS_ANDROID)
+bool RegisterThreadUtils(JNIEnv* env) {
+  return RegisterNativesImpl(env);
+}
+
+void PlatformThread::SetThreadPriority(PlatformThreadHandle,
+                                       ThreadPriority priority) {
+  if (priority == kThreadPriority_RealtimeAudio) {
+    JNIEnv* env = base::android::AttachCurrentThread();
+    Java_ThreadUtils_setThreadPriorityAudio(env, PlatformThread::CurrentId());
+  } else {
+    NOTREACHED() << "Unknown thread priority.";
+  }
+}
+#endif // defined(OS_ANDROID)

[Mark Mentovai, 2013/01/23 13:59:19]

Nit: this needs another space between #endif and //.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Implem...
(“Line Comments”)

[Wei James(wistoch), 2013/01/24 00:41:59]

fixed. thanks 

+
 }  // namespace base