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

Side by Side Diff: src/libplatform/default-platform.cc

Issue 510693003: Sync our homegrown SysInfo replacement with the one in Chrome base. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Add missing include. Created 6 years, 3 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/isolate.cc ('k') | test/base-unittests/base-unittests.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/libplatform/default-platform.h" 5 #include "src/libplatform/default-platform.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <queue> 8 #include <queue>
9 9
10 #include "src/base/logging.h" 10 #include "src/base/logging.h"
11 #include "src/base/platform/platform.h" 11 #include "src/base/platform/platform.h"
12 #include "src/base/sys-info.h"
12 #include "src/libplatform/worker-thread.h" 13 #include "src/libplatform/worker-thread.h"
13 14
14 namespace v8 { 15 namespace v8 {
15 namespace platform { 16 namespace platform {
16 17
17 18
18 v8::Platform* CreateDefaultPlatform(int thread_pool_size) { 19 v8::Platform* CreateDefaultPlatform(int thread_pool_size) {
19 DefaultPlatform* platform = new DefaultPlatform(); 20 DefaultPlatform* platform = new DefaultPlatform();
20 platform->SetThreadPoolSize(thread_pool_size); 21 platform->SetThreadPoolSize(thread_pool_size);
21 platform->EnsureInitialized(); 22 platform->EnsureInitialized();
(...skipping 29 matching lines...) Expand all
51 delete i->second.front(); 52 delete i->second.front();
52 i->second.pop(); 53 i->second.pop();
53 } 54 }
54 } 55 }
55 } 56 }
56 57
57 58
58 void DefaultPlatform::SetThreadPoolSize(int thread_pool_size) { 59 void DefaultPlatform::SetThreadPoolSize(int thread_pool_size) {
59 base::LockGuard<base::Mutex> guard(&lock_); 60 base::LockGuard<base::Mutex> guard(&lock_);
60 DCHECK(thread_pool_size >= 0); 61 DCHECK(thread_pool_size >= 0);
61 if (thread_pool_size < 1) 62 if (thread_pool_size < 1) {
62 thread_pool_size = base::OS::NumberOfProcessorsOnline(); 63 thread_pool_size = base::SysInfo::NumberOfProcessors();
64 }
63 thread_pool_size_ = 65 thread_pool_size_ =
64 std::max(std::min(thread_pool_size, kMaxThreadPoolSize), 1); 66 std::max(std::min(thread_pool_size, kMaxThreadPoolSize), 1);
65 } 67 }
66 68
67 69
68 void DefaultPlatform::EnsureInitialized() { 70 void DefaultPlatform::EnsureInitialized() {
69 base::LockGuard<base::Mutex> guard(&lock_); 71 base::LockGuard<base::Mutex> guard(&lock_);
70 if (initialized_) return; 72 if (initialized_) return;
71 initialized_ = true; 73 initialized_ = true;
72 74
(...skipping 25 matching lines...) Expand all
98 queue_.Append(task); 100 queue_.Append(task);
99 } 101 }
100 102
101 103
102 void DefaultPlatform::CallOnForegroundThread(v8::Isolate* isolate, Task* task) { 104 void DefaultPlatform::CallOnForegroundThread(v8::Isolate* isolate, Task* task) {
103 base::LockGuard<base::Mutex> guard(&lock_); 105 base::LockGuard<base::Mutex> guard(&lock_);
104 main_thread_queue_[isolate].push(task); 106 main_thread_queue_[isolate].push(task);
105 } 107 }
106 108
107 } } // namespace v8::platform 109 } } // namespace v8::platform
OLDNEW
« no previous file with comments | « src/isolate.cc ('k') | test/base-unittests/base-unittests.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698