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

Unified Diff: base/synchronization/waitable_event.h

Issue 2030683005: Remove boolean-based WaitableEvent constructor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@chrome_thread_watcher
Patch Set: Created 4 years, 6 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: base/synchronization/waitable_event.h
diff --git a/base/synchronization/waitable_event.h b/base/synchronization/waitable_event.h
index a4ca312174087b4f2a9aec34ebcf31f5507012ef..3863e98455ee10e74f1ee6c40994aadd5012d947 100644
--- a/base/synchronization/waitable_event.h
+++ b/base/synchronization/waitable_event.h
@@ -56,14 +56,6 @@ class BASE_EXPORT WaitableEvent {
// the above enums.
WaitableEvent(ResetPolicy reset_policy, InitialState initial_state);
- // If manual_reset is true, then to set the event state to non-signaled, a
- // consumer must call the Reset method. If this parameter is false, then the
- // system automatically resets the event state to non-signaled after a single
- // waiting thread has been released.
- // DEPRECATED: Use the enum-based constructor instead (full removal tracked in
- // http://crbug.com/612843).
- WaitableEvent(bool manual_reset, bool initially_signaled);
-
#if defined(OS_WIN)
// Create a WaitableEvent from an Event HANDLE which has already been
// created. This objects takes ownership of the HANDLE and will close it when
@@ -165,7 +157,7 @@ class BASE_EXPORT WaitableEvent {
struct WaitableEventKernel :
public RefCountedThreadSafe<WaitableEventKernel> {
public:
- WaitableEventKernel(bool manual_reset, bool initially_signaled);
+ WaitableEventKernel(ResetPolicy reset_policy, InitialState initial_state);
bool Dequeue(Waiter* waiter, void* tag);
« no previous file with comments | « no previous file | base/synchronization/waitable_event_posix.cc » ('j') | base/synchronization/waitable_event_posix.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698