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

Unified Diff: src/base/platform/platform-win32.cc

Issue 1118533003: Make CPU profiler do not hog 100% of CPU. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Do not use Sleep on Windows Created 5 years, 8 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: src/base/platform/platform-win32.cc
diff --git a/src/base/platform/platform-win32.cc b/src/base/platform/platform-win32.cc
index b0677afad55cb84545ac92989196db1522985c3f..3226be7ea9479c8f6143b79585bd20d74c72629c 100644
--- a/src/base/platform/platform-win32.cc
+++ b/src/base/platform/platform-win32.cc
@@ -812,8 +812,8 @@ void OS::Guard(void* address, const size_t size) {
}
-void OS::Sleep(int milliseconds) {
- ::Sleep(milliseconds);
+void OS::Sleep(TimeDelta interval) {
+ ::Sleep(static_cast<DWORD>(interval.InMilliseconds()));
}
@@ -1385,9 +1385,5 @@ void Thread::SetThreadLocal(LocalStorageKey key, void* value) {
}
-
-void Thread::YieldCPU() {
- Sleep(0);
-}
-
+void Thread::YieldCPU() { ::Sleep(0); }
} } // namespace v8::base
« no previous file with comments | « src/base/platform/platform-posix.cc ('k') | src/cpu-profiler.cc » ('j') | src/cpu-profiler.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698