Commit 44f1f742 authored by Francois Doray's avatar Francois Doray Committed by Commit Bot

[base] Don't DCHECK GetCurrentThreadPriority() at the end of SetCurrentThreadPriority().

The DCHECK passes most of the time, but not always. It is inherently
racy, as external code could change the thread priority between when
we set it and when we DCHECK it. Removing it as the failures are
not actionnable.

Bug: 1044882
Change-Id: I0adb7a801c10fb15ada3ed229175fb08c0232b8f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2031705Reviewed-by: default avatarGabriel Charette <gab@chromium.org>
Commit-Queue: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#737483}
parent a93b363c
...@@ -384,8 +384,6 @@ void PlatformThread::SetCurrentThreadPriorityImpl(ThreadPriority priority) { ...@@ -384,8 +384,6 @@ void PlatformThread::SetCurrentThreadPriorityImpl(ThreadPriority priority) {
internal::AssertMemoryPriority(thread_handle, MEMORY_PRIORITY_VERY_LOW); internal::AssertMemoryPriority(thread_handle, MEMORY_PRIORITY_VERY_LOW);
} }
} }
DCHECK_EQ(GetCurrentThreadPriority(), priority);
} }
// static // static
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment