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

Side by Side Diff: src/libplatform/task-queue.cc

Issue 430503007: Rename ASSERT* to DCHECK*. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: REBASE and fixes Created 6 years, 4 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/libplatform/default-platform.cc ('k') | src/list.h » ('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/task-queue.h" 5 #include "src/libplatform/task-queue.h"
6 6
7 #include "src/base/logging.h" 7 #include "src/base/logging.h"
8 8
9 namespace v8 { 9 namespace v8 {
10 namespace platform { 10 namespace platform {
11 11
12 TaskQueue::TaskQueue() : process_queue_semaphore_(0), terminated_(false) {} 12 TaskQueue::TaskQueue() : process_queue_semaphore_(0), terminated_(false) {}
13 13
14 14
15 TaskQueue::~TaskQueue() { 15 TaskQueue::~TaskQueue() {
16 base::LockGuard<base::Mutex> guard(&lock_); 16 base::LockGuard<base::Mutex> guard(&lock_);
17 ASSERT(terminated_); 17 DCHECK(terminated_);
18 ASSERT(task_queue_.empty()); 18 DCHECK(task_queue_.empty());
19 } 19 }
20 20
21 21
22 void TaskQueue::Append(Task* task) { 22 void TaskQueue::Append(Task* task) {
23 base::LockGuard<base::Mutex> guard(&lock_); 23 base::LockGuard<base::Mutex> guard(&lock_);
24 ASSERT(!terminated_); 24 DCHECK(!terminated_);
25 task_queue_.push(task); 25 task_queue_.push(task);
26 process_queue_semaphore_.Signal(); 26 process_queue_semaphore_.Signal();
27 } 27 }
28 28
29 29
30 Task* TaskQueue::GetNext() { 30 Task* TaskQueue::GetNext() {
31 for (;;) { 31 for (;;) {
32 { 32 {
33 base::LockGuard<base::Mutex> guard(&lock_); 33 base::LockGuard<base::Mutex> guard(&lock_);
34 if (!task_queue_.empty()) { 34 if (!task_queue_.empty()) {
35 Task* result = task_queue_.front(); 35 Task* result = task_queue_.front();
36 task_queue_.pop(); 36 task_queue_.pop();
37 return result; 37 return result;
38 } 38 }
39 if (terminated_) { 39 if (terminated_) {
40 process_queue_semaphore_.Signal(); 40 process_queue_semaphore_.Signal();
41 return NULL; 41 return NULL;
42 } 42 }
43 } 43 }
44 process_queue_semaphore_.Wait(); 44 process_queue_semaphore_.Wait();
45 } 45 }
46 } 46 }
47 47
48 48
49 void TaskQueue::Terminate() { 49 void TaskQueue::Terminate() {
50 base::LockGuard<base::Mutex> guard(&lock_); 50 base::LockGuard<base::Mutex> guard(&lock_);
51 ASSERT(!terminated_); 51 DCHECK(!terminated_);
52 terminated_ = true; 52 terminated_ = true;
53 process_queue_semaphore_.Signal(); 53 process_queue_semaphore_.Signal();
54 } 54 }
55 55
56 } } // namespace v8::platform 56 } } // namespace v8::platform
OLDNEW
« no previous file with comments | « src/libplatform/default-platform.cc ('k') | src/list.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698