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

Side by Side Diff: base/synchronization/condition_variable.h

Issue 403803004: base: Enable lock dchecks with dchecks_always_on (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix condition variable too Created 6 years, 5 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
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium 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 // ConditionVariable wraps pthreads condition variable synchronization or, on 5 // ConditionVariable wraps pthreads condition variable synchronization or, on
6 // Windows, simulates it. This functionality is very helpful for having 6 // Windows, simulates it. This functionality is very helpful for having
7 // several threads wait for an event, as is common with a thread pool managed 7 // several threads wait for an event, as is common with a thread pool managed
8 // by a master. The meaning of such an event in the (worker) thread pool 8 // by a master. The meaning of such an event in the (worker) thread pool
9 // scenario is that additional tasks are now available for processing. It is 9 // scenario is that additional tasks are now available for processing. It is
10 // used in Chrome in the DNS prefetching system to notify worker threads that 10 // used in Chrome in the DNS prefetching system to notify worker threads that
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 // Signal() revives one waiting thread. 97 // Signal() revives one waiting thread.
98 void Signal(); 98 void Signal();
99 99
100 private: 100 private:
101 101
102 #if defined(OS_WIN) 102 #if defined(OS_WIN)
103 ConditionVarImpl* impl_; 103 ConditionVarImpl* impl_;
104 #elif defined(OS_POSIX) 104 #elif defined(OS_POSIX)
105 pthread_cond_t condition_; 105 pthread_cond_t condition_;
106 pthread_mutex_t* user_mutex_; 106 pthread_mutex_t* user_mutex_;
107 #if !defined(NDEBUG) 107 #if !defined(NDEBUG) || defined(DCHECK_ALWAYS_ON)
108 base::Lock* user_lock_; // Needed to adjust shadow lock state on wait. 108 base::Lock* user_lock_; // Needed to adjust shadow lock state on wait.
109 #endif 109 #endif
110 110
111 #endif 111 #endif
112 112
113 DISALLOW_COPY_AND_ASSIGN(ConditionVariable); 113 DISALLOW_COPY_AND_ASSIGN(ConditionVariable);
114 }; 114 };
115 115
116 } // namespace base 116 } // namespace base
117 117
118 #endif // BASE_SYNCHRONIZATION_CONDITION_VARIABLE_H_ 118 #endif // BASE_SYNCHRONIZATION_CONDITION_VARIABLE_H_
OLDNEW
« no previous file with comments | « no previous file | base/synchronization/condition_variable_posix.cc » ('j') | base/synchronization/lock.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698