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

Side by Side Diff: mojo/edk/system/simple_dispatcher.h

Issue 1350023003: Add a Mojo EDK for Chrome that uses one OS pipe per message pipe. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: more cleanup Created 5 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 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 #ifndef THIRD_PARTY_MOJO_SRC_MOJO_EDK_SYSTEM_SIMPLE_DISPATCHER_H_ 5 #ifndef MOJO_EDK_SYSTEM_SIMPLE_DISPATCHER_H_
6 #define THIRD_PARTY_MOJO_SRC_MOJO_EDK_SYSTEM_SIMPLE_DISPATCHER_H_ 6 #define MOJO_EDK_SYSTEM_SIMPLE_DISPATCHER_H_
7 7
8 #include <list> 8 #include <list>
9 9
10 #include "mojo/edk/system/awakable_list.h"
11 #include "mojo/edk/system/dispatcher.h"
12 #include "mojo/edk/system/system_impl_export.h"
10 #include "mojo/public/cpp/system/macros.h" 13 #include "mojo/public/cpp/system/macros.h"
11 #include "third_party/mojo/src/mojo/edk/system/awakable_list.h"
12 #include "third_party/mojo/src/mojo/edk/system/dispatcher.h"
13 #include "third_party/mojo/src/mojo/edk/system/system_impl_export.h"
14 14
15 namespace mojo { 15 namespace mojo {
16 namespace system { 16 namespace edk {
17 17
18 // A base class for simple dispatchers. "Simple" means that there's a one-to-one 18 // A base class for simple dispatchers. "Simple" means that there's a one-to-one
19 // correspondence between handles and dispatchers (see the explanatory comment 19 // correspondence between handles and dispatchers (see the explanatory comment
20 // in core.cc). This class implements the standard waiter-signalling mechanism 20 // in core.cc). This class implements the standard waiter-signalling mechanism
21 // in that case. 21 // in that case.
22 class MOJO_SYSTEM_IMPL_EXPORT SimpleDispatcher : public Dispatcher { 22 class MOJO_SYSTEM_IMPL_EXPORT SimpleDispatcher : public Dispatcher {
23 protected: 23 protected:
24 SimpleDispatcher(); 24 SimpleDispatcher();
25 ~SimpleDispatcher() override; 25 ~SimpleDispatcher() override;
26 26
27 // To be called by subclasses when the state changes (so 27 // To be called by subclasses when the state changes (so
28 // |GetHandleSignalsStateImplNoLock()| should be checked again). 28 // |GetHandleSignalsStateImplNoLock()| should be checked again). Must be
29 void HandleSignalsStateChangedNoLock() MOJO_EXCLUSIVE_LOCKS_REQUIRED(mutex()); 29 // called under lock.
30 void HandleSignalsStateChangedNoLock();
30 31
31 // |Dispatcher| protected methods: 32 // |Dispatcher| protected methods:
32 void CancelAllAwakablesNoLock() override; 33 void CancelAllAwakablesNoLock() override;
33 MojoResult AddAwakableImplNoLock(Awakable* awakable, 34 MojoResult AddAwakableImplNoLock(Awakable* awakable,
34 MojoHandleSignals signals, 35 MojoHandleSignals signals,
35 uint32_t context, 36 uint32_t context,
36 HandleSignalsState* signals_state) override; 37 HandleSignalsState* signals_state) override;
37 void RemoveAwakableImplNoLock(Awakable* awakable, 38 void RemoveAwakableImplNoLock(Awakable* awakable,
38 HandleSignalsState* signals_state) override; 39 HandleSignalsState* signals_state) override;
39 40
40 private: 41 private:
41 AwakableList awakable_list_ MOJO_GUARDED_BY(mutex()); 42 // Protected by |lock()|:
43 AwakableList awakable_list_;
42 44
43 MOJO_DISALLOW_COPY_AND_ASSIGN(SimpleDispatcher); 45 MOJO_DISALLOW_COPY_AND_ASSIGN(SimpleDispatcher);
44 }; 46 };
45 47
46 } // namespace system 48 } // namespace edk
47 } // namespace mojo 49 } // namespace mojo
48 50
49 #endif // THIRD_PARTY_MOJO_SRC_MOJO_EDK_SYSTEM_SIMPLE_DISPATCHER_H_ 51 #endif // MOJO_EDK_SYSTEM_SIMPLE_DISPATCHER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698