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

Side by Side Diff: base/condition_variable.h

Issue 43148: Remove unneeded uses of logging.h in header files. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 9 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 | « no previous file | base/non_thread_safe.h » ('j') | chrome/common/chrome_plugin_util.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 // may improve performance, as the selected thread may have a greater chance of 59 // may improve performance, as the selected thread may have a greater chance of
60 // having some of its stack data in various CPU caches. 60 // having some of its stack data in various CPU caches.
61 // 61 //
62 // For a discussion of the many very subtle implementation details, see the FAQ 62 // For a discussion of the many very subtle implementation details, see the FAQ
63 // at the end of condition_variable_win.cc. 63 // at the end of condition_variable_win.cc.
64 64
65 #ifndef BASE_CONDITION_VARIABLE_H_ 65 #ifndef BASE_CONDITION_VARIABLE_H_
66 #define BASE_CONDITION_VARIABLE_H_ 66 #define BASE_CONDITION_VARIABLE_H_
67 67
68 #include "base/lock.h" 68 #include "base/lock.h"
69 #include "base/logging.h"
70 #include "base/scoped_ptr.h" 69 #include "base/scoped_ptr.h"
71 #include "base/time.h" 70 #include "base/time.h"
72 71
73 class ConditionVariable { 72 class ConditionVariable {
74 public: 73 public:
75 // Construct a cv for use with ONLY one user lock. 74 // Construct a cv for use with ONLY one user lock.
76 explicit ConditionVariable(Lock* user_lock); 75 explicit ConditionVariable(Lock* user_lock);
77 76
78 ~ConditionVariable(); 77 ~ConditionVariable();
79 78
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 163
165 pthread_cond_t condition_; 164 pthread_cond_t condition_;
166 pthread_mutex_t* user_mutex_; 165 pthread_mutex_t* user_mutex_;
167 166
168 #endif 167 #endif
169 168
170 DISALLOW_COPY_AND_ASSIGN(ConditionVariable); 169 DISALLOW_COPY_AND_ASSIGN(ConditionVariable);
171 }; 170 };
172 171
173 #endif // BASE_CONDITION_VARIABLE_H_ 172 #endif // BASE_CONDITION_VARIABLE_H_
OLDNEW
« no previous file with comments | « no previous file | base/non_thread_safe.h » ('j') | chrome/common/chrome_plugin_util.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698