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

Side by Side Diff: mojo/system/waiter_list.cc

Issue 325213004: Mojo: Wrap the satisfied/unsatisfied wait flags state in a single object. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « mojo/system/waiter_list.h ('k') | mojo/system/waiter_list_unittest.cc » ('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 Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #include "mojo/system/waiter_list.h" 5 #include "mojo/system/waiter_list.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "mojo/system/wait_flags_state.h"
8 #include "mojo/system/waiter.h" 9 #include "mojo/system/waiter.h"
9 10
10 namespace mojo { 11 namespace mojo {
11 namespace system { 12 namespace system {
12 13
13 WaiterList::WaiterList() { 14 WaiterList::WaiterList() {
14 } 15 }
15 16
16 WaiterList::~WaiterList() { 17 WaiterList::~WaiterList() {
17 DCHECK(waiters_.empty()); 18 DCHECK(waiters_.empty());
18 } 19 }
19 20
21 void WaiterList::AwakeWaitersForStateChange(const WaitFlagsState& state) {
22 for (WaiterInfoList::iterator it = waiters_.begin(); it != waiters_.end();
23 ++it) {
24 if (state.satisfies(it->flags))
25 it->waiter->Awake(it->wake_result);
26 else if (!state.can_satisfy(it->flags))
27 it->waiter->Awake(MOJO_RESULT_FAILED_PRECONDITION);
28 }
29 }
30
31 //FIXME Remove:
20 void WaiterList::AwakeWaitersForStateChange(MojoWaitFlags satisfied_flags, 32 void WaiterList::AwakeWaitersForStateChange(MojoWaitFlags satisfied_flags,
21 MojoWaitFlags satisfiable_flags) { 33 MojoWaitFlags satisfiable_flags) {
22 for (WaiterInfoList::iterator it = waiters_.begin(); it != waiters_.end(); 34 for (WaiterInfoList::iterator it = waiters_.begin(); it != waiters_.end();
23 ++it) { 35 ++it) {
24 if (it->flags & satisfied_flags) 36 if (it->flags & satisfied_flags)
25 it->waiter->Awake(it->wake_result); 37 it->waiter->Awake(it->wake_result);
26 else if (!(it->flags & satisfiable_flags)) 38 else if (!(it->flags & satisfiable_flags))
27 it->waiter->Awake(MOJO_RESULT_FAILED_PRECONDITION); 39 it->waiter->Awake(MOJO_RESULT_FAILED_PRECONDITION);
28 } 40 }
29 } 41 }
(...skipping 18 matching lines...) Expand all
48 for (WaiterInfoList::iterator it = waiters_.begin(); it != waiters_.end();) { 60 for (WaiterInfoList::iterator it = waiters_.begin(); it != waiters_.end();) {
49 WaiterInfoList::iterator maybe_delete = it; 61 WaiterInfoList::iterator maybe_delete = it;
50 ++it; 62 ++it;
51 if (maybe_delete->waiter == waiter) 63 if (maybe_delete->waiter == waiter)
52 waiters_.erase(maybe_delete); 64 waiters_.erase(maybe_delete);
53 } 65 }
54 } 66 }
55 67
56 } // namespace system 68 } // namespace system
57 } // namespace mojo 69 } // namespace mojo
OLDNEW
« no previous file with comments | « mojo/system/waiter_list.h ('k') | mojo/system/waiter_list_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698