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

Side by Side Diff: cc/completion_event.h

Issue 11048044: cc: Switch to Chromium DCHECKs and LOGs (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: rebase Created 8 years, 2 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
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 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 #ifndef CCCompletionEvent_h 5 #ifndef CCCompletionEvent_h
6 #define CCCompletionEvent_h 6 #define CCCompletionEvent_h
7 7
8 #include "base/synchronization/waitable_event.h" 8 #include "base/synchronization/waitable_event.h"
9 #include "cc/dcheck.h"
9 10
10 namespace cc { 11 namespace cc {
11 12
12 // Used for making blocking calls from one thread to another. Use only when 13 // Used for making blocking calls from one thread to another. Use only when
13 // absolutely certain that doing-so will not lead to a deadlock. 14 // absolutely certain that doing-so will not lead to a deadlock.
14 // 15 //
15 // It is safe to destroy this object as soon as wait() returns. 16 // It is safe to destroy this object as soon as wait() returns.
16 class CCCompletionEvent { 17 class CCCompletionEvent {
17 public: 18 public:
18 CCCompletionEvent() 19 CCCompletionEvent()
19 : m_event(false /* manual_reset */, false /* initially_signaled */) 20 : m_event(false /* manual_reset */, false /* initially_signaled */)
20 { 21 {
21 #ifndef NDEBUG 22 #if CC_DCHECK_ENABLED
22 m_waited = false; 23 m_waited = false;
23 m_signaled = false; 24 m_signaled = false;
24 #endif 25 #endif
25 } 26 }
26 27
27 ~CCCompletionEvent() 28 ~CCCompletionEvent()
28 { 29 {
29 ASSERT(m_waited); 30 CC_DCHECK(m_waited);
30 ASSERT(m_signaled); 31 CC_DCHECK(m_signaled);
31 } 32 }
32 33
33 void wait() 34 void wait()
34 { 35 {
35 ASSERT(!m_waited); 36 CC_DCHECK(!m_waited);
36 #ifndef NDEBUG 37 #if CC_DCHECK_ENABLED
37 m_waited = true; 38 m_waited = true;
38 #endif 39 #endif
39 m_event.Wait(); 40 m_event.Wait();
40 } 41 }
41 42
42 void signal() 43 void signal()
43 { 44 {
44 ASSERT(!m_signaled); 45 CC_DCHECK(!m_signaled);
45 #ifndef NDEBUG 46 #if CC_DCHECK_ENABLED
46 m_signaled = true; 47 m_signaled = true;
47 #endif 48 #endif
48 m_event.Signal(); 49 m_event.Signal();
49 } 50 }
50 51
51 private: 52 private:
52 base::WaitableEvent m_event; 53 base::WaitableEvent m_event;
53 #ifndef NDEBUG 54 #if CC_DCHECK_ENABLED
54 // Used to assert that wait() and signal() are each called exactly once. 55 // Used to assert that wait() and signal() are each called exactly once.
55 bool m_waited; 56 bool m_waited;
56 bool m_signaled; 57 bool m_signaled;
57 #endif 58 #endif
58 }; 59 };
59 60
60 } 61 }
61 62
62 #endif 63 #endif
OLDNEW
« no previous file with comments | « cc/checkerboard_draw_quad.cc ('k') | cc/damage_tracker.cc » ('j') | cc/yuv_video_draw_quad.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698