Add full SetThreadPriority support to Windows.

base/threading/platform_thread_win.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 "base/debug/alias.h"
 #include "base/debug/profiler.h"
 #include "base/logging.h"
 #include "base/threading/thread_id_name_manager.h"
@@ skipping 108 matching lines @@
     *out_thread_handle = PlatformThreadHandle(thread_handle);
   else
     CloseHandle(thread_handle);
   return true;
 }
 
 }  // namespace
 
 // static
 PlatformThreadId PlatformThread::CurrentId() {
-  return GetCurrentThreadId();
+  return ::GetCurrentThreadId();
 }
 
 // static
 PlatformThreadRef PlatformThread::CurrentRef() {
-  return PlatformThreadRef(GetCurrentThreadId());
+  return PlatformThreadRef(::GetCurrentThreadId());
 }
 
 // static
 PlatformThreadHandle PlatformThread::CurrentHandle() {
-  NOTIMPLEMENTED(); // See OpenThread()
-  return PlatformThreadHandle();
+  return PlatformThreadHandle(::GetCurrentThread());
 }
 
 // static
 void PlatformThread::YieldCurrentThread() {
   ::Sleep(0);
 }
 
 // static
 void PlatformThread::Sleep(TimeDelta duration) {
   // When measured with a high resolution clock, Sleep() sometimes returns much
@@ skipping 76 matching lines @@
     debug::Alias(&thread_handle.handle_);
     CHECK(false);
   }
 
   CloseHandle(thread_handle.handle_);
 }
 
 // static
 void PlatformThread::SetThreadPriority(PlatformThreadHandle handle,
                                        ThreadPriority priority) {
-  switch (priority) {
+  DCHECK(!handle.is_null());
+
+  int desired_priority = THREAD_PRIORITY_ERROR_RETURN;
+  switch (desired_priority) {
     case kThreadPriority_Normal:
-      ::SetThreadPriority(handle.handle_, THREAD_PRIORITY_NORMAL);
+      desired_priority = THREAD_PRIORITY_NORMAL;
       break;
     case kThreadPriority_RealtimeAudio:
-      ::SetThreadPriority(handle.handle_, THREAD_PRIORITY_TIME_CRITICAL);
+      desired_priority = THREAD_PRIORITY_TIME_CRITICAL;
+      break;
+    case kThreadPriority_Display:
+      desired_priority = THREAD_PRIORITY_ABOVE_NORMAL;
+      break;
+    case kThreadPriority_Background:
+      desired_priority = THREAD_PRIORITY_LOWEST;
       break;
     default:
       NOTREACHED() << "Unknown priority.";
       break;
   }
+  DCHECK_NE(desired_priority, THREAD_PRIORITY_ERROR_RETURN);

[rvargas (doing something else), 2015/03/30 22:31:23]

Isn't this the same as line 253?

[gab, 2015/03/31 14:02:33]

If all case statements set |desired_priority|, yes, this is just explicitly
stating the contract that they must.

+
+  BOOL success = ::SetThreadPriority(handle.handle_, desired_priority);
+  DPCHECK(success);
+}
+
+// static
+ThreadPriority PlatformThread::GetThreadPriority(PlatformThreadHandle handle) {
+  DCHECK(!handle.is_null());
+
+  int priority = ::GetThreadPriority(handle.handle_);
+  switch (priority) {
+    case THREAD_PRIORITY_NORMAL:
+      return kThreadPriority_Normal;
+    case THREAD_PRIORITY_TIME_CRITICAL:
+      return kThreadPriority_RealtimeAudio;
+    case THREAD_PRIORITY_ABOVE_NORMAL:
+      return kThreadPriority_Display;
+    case THREAD_PRIORITY_LOWEST:
+      return kThreadPriority_Background;
+    case THREAD_PRIORITY_ERROR_RETURN:
+      DPCHECK(false) << "GetThreadPriority error";  // Falls through.
+    default:
+      NOTREACHED() << "Unexpected priority: " << priority;
+      return kThreadPriority_Normal;
+  }
 }
 
 }  // namespace base