[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 1 | // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
license.bot | bf09a50 | 2008-08-24 00:55:55 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 4 | |
[email protected] | 946d1b2 | 2009-07-22 23:57:21 | [diff] [blame] | 5 | #include "ipc/ipc_sync_channel.h" |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 6 | |
[email protected] | 72b6f8e2 | 2011-11-12 21:16:41 | [diff] [blame] | 7 | #include "base/bind.h" |
[email protected] | f886b7bf | 2008-09-10 10:54:06 | [diff] [blame] | 8 | #include "base/lazy_instance.h" |
[email protected] | c62dd9d | 2011-09-21 18:05:41 | [diff] [blame] | 9 | #include "base/location.h" |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 10 | #include "base/logging.h" |
[email protected] | 44f9c95 | 2011-01-02 06:05:39 | [diff] [blame] | 11 | #include "base/synchronization/waitable_event.h" |
| 12 | #include "base/synchronization/waitable_event_watcher.h" |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 13 | #include "base/thread_task_runner_handle.h" |
| 14 | #include "base/threading/thread_local.h" |
[email protected] | 946d1b2 | 2009-07-22 23:57:21 | [diff] [blame] | 15 | #include "ipc/ipc_sync_message.h" |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 16 | |
[email protected] | e1acf6f | 2008-10-27 20:43:33 | [diff] [blame] | 17 | using base::TimeDelta; |
| 18 | using base::TimeTicks; |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 19 | using base::WaitableEvent; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 20 | |
| 21 | namespace IPC { |
| 22 | // When we're blocked in a Send(), we need to process incoming synchronous |
| 23 | // messages right away because it could be blocking our reply (either |
| 24 | // directly from the same object we're calling, or indirectly through one or |
| 25 | // more other channels). That means that in SyncContext's OnMessageReceived, |
| 26 | // we need to process sync message right away if we're blocked. However a |
| 27 | // simple check isn't sufficient, because the listener thread can be in the |
| 28 | // process of calling Send. |
| 29 | // To work around this, when SyncChannel filters a sync message, it sets |
| 30 | // an event that the listener thread waits on during its Send() call. This |
| 31 | // allows us to dispatch incoming sync messages when blocked. The race |
| 32 | // condition is handled because if Send is in the process of being called, it |
| 33 | // will check the event. In case the listener thread isn't sending a message, |
| 34 | // we queue a task on the listener thread to dispatch the received messages. |
| 35 | // The messages are stored in this queue object that's shared among all |
| 36 | // SyncChannel objects on the same thread (since one object can receive a |
| 37 | // sync message while another one is blocked). |
| 38 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 39 | class SyncChannel::ReceivedSyncMsgQueue : |
| 40 | public base::RefCountedThreadSafe<ReceivedSyncMsgQueue> { |
| 41 | public: |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 42 | // Returns the ReceivedSyncMsgQueue instance for this thread, creating one |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 43 | // if necessary. Call RemoveContext on the same thread when done. |
| 44 | static ReceivedSyncMsgQueue* AddContext() { |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 45 | // We want one ReceivedSyncMsgQueue per listener thread (i.e. since multiple |
| 46 | // SyncChannel objects can block the same thread). |
| 47 | ReceivedSyncMsgQueue* rv = lazy_tls_ptr_.Pointer()->Get(); |
| 48 | if (!rv) { |
| 49 | rv = new ReceivedSyncMsgQueue(); |
| 50 | ReceivedSyncMsgQueue::lazy_tls_ptr_.Pointer()->Set(rv); |
| 51 | } |
| 52 | rv->listener_count_++; |
| 53 | return rv; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 54 | } |
| 55 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 56 | // Called on IPC thread when a synchronous message or reply arrives. |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 57 | void QueueMessage(const Message& msg, SyncChannel::SyncContext* context) { |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 58 | bool was_task_pending; |
| 59 | { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 60 | base::AutoLock auto_lock(message_lock_); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 61 | |
| 62 | was_task_pending = task_pending_; |
| 63 | task_pending_ = true; |
| 64 | |
| 65 | // We set the event in case the listener thread is blocked (or is about |
| 66 | // to). In case it's not, the PostTask dispatches the messages. |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 67 | message_queue_.push_back(QueuedMessage(new Message(msg), context)); |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 68 | message_queue_version_++; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 69 | } |
| 70 | |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 71 | dispatch_event_.Signal(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 72 | if (!was_task_pending) { |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 73 | listener_task_runner_->PostTask( |
[email protected] | 72b6f8e2 | 2011-11-12 21:16:41 | [diff] [blame] | 74 | FROM_HERE, base::Bind(&ReceivedSyncMsgQueue::DispatchMessagesTask, |
| 75 | this, scoped_refptr<SyncContext>(context))); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 76 | } |
| 77 | } |
| 78 | |
| 79 | void QueueReply(const Message &msg, SyncChannel::SyncContext* context) { |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 80 | received_replies_.push_back(QueuedMessage(new Message(msg), context)); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 81 | } |
| 82 | |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 83 | // Called on the listener's thread to process any queues synchronous |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 84 | // messages. |
[email protected] | 54af05f | 2011-04-08 03:38:21 | [diff] [blame] | 85 | void DispatchMessagesTask(SyncContext* context) { |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 86 | { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 87 | base::AutoLock auto_lock(message_lock_); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 88 | task_pending_ = false; |
| 89 | } |
[email protected] | 54af05f | 2011-04-08 03:38:21 | [diff] [blame] | 90 | context->DispatchMessages(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 91 | } |
| 92 | |
[email protected] | 54af05f | 2011-04-08 03:38:21 | [diff] [blame] | 93 | void DispatchMessages(SyncContext* dispatching_context) { |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 94 | bool first_time = true; |
| 95 | uint32 expected_version = 0; |
| 96 | SyncMessageQueue::iterator it; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 97 | while (true) { |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 98 | Message* message = NULL; |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 99 | scoped_refptr<SyncChannel::SyncContext> context; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 100 | { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 101 | base::AutoLock auto_lock(message_lock_); |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 102 | if (first_time || message_queue_version_ != expected_version) { |
| 103 | it = message_queue_.begin(); |
| 104 | first_time = false; |
[email protected] | 54af05f | 2011-04-08 03:38:21 | [diff] [blame] | 105 | } |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 106 | for (; it != message_queue_.end(); it++) { |
[email protected] | 298ee7d | 2012-03-30 21:29:30 | [diff] [blame] | 107 | int message_group = it->context->restrict_dispatch_group(); |
| 108 | if (message_group == kRestrictDispatchGroup_None || |
| 109 | message_group == dispatching_context->restrict_dispatch_group()) { |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 110 | message = it->message; |
| 111 | context = it->context; |
| 112 | it = message_queue_.erase(it); |
| 113 | message_queue_version_++; |
| 114 | expected_version = message_queue_version_; |
| 115 | break; |
| 116 | } |
| 117 | } |
| 118 | } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 119 | |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 120 | if (message == NULL) |
| 121 | break; |
| 122 | context->OnDispatchMessage(*message); |
| 123 | delete message; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 124 | } |
| 125 | } |
| 126 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 127 | // SyncChannel calls this in its destructor. |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 128 | void RemoveContext(SyncContext* context) { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 129 | base::AutoLock auto_lock(message_lock_); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 130 | |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 131 | SyncMessageQueue::iterator iter = message_queue_.begin(); |
| 132 | while (iter != message_queue_.end()) { |
| 133 | if (iter->context == context) { |
| 134 | delete iter->message; |
| 135 | iter = message_queue_.erase(iter); |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 136 | message_queue_version_++; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 137 | } else { |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 138 | iter++; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 139 | } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 140 | } |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 141 | |
| 142 | if (--listener_count_ == 0) { |
| 143 | DCHECK(lazy_tls_ptr_.Pointer()->Get()); |
| 144 | lazy_tls_ptr_.Pointer()->Set(NULL); |
| 145 | } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 146 | } |
| 147 | |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 148 | WaitableEvent* dispatch_event() { return &dispatch_event_; } |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 149 | base::SingleThreadTaskRunner* listener_task_runner() { |
| 150 | return listener_task_runner_; |
[email protected] | 92bf906 | 2011-05-02 18:00:49 | [diff] [blame] | 151 | } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 152 | |
[email protected] | f886b7bf | 2008-09-10 10:54:06 | [diff] [blame] | 153 | // Holds a pointer to the per-thread ReceivedSyncMsgQueue object. |
| 154 | static base::LazyInstance<base::ThreadLocalPointer<ReceivedSyncMsgQueue> > |
| 155 | lazy_tls_ptr_; |
| 156 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 157 | // Called on the ipc thread to check if we can unblock any current Send() |
| 158 | // calls based on a queued reply. |
| 159 | void DispatchReplies() { |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 160 | for (size_t i = 0; i < received_replies_.size(); ++i) { |
| 161 | Message* message = received_replies_[i].message; |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 162 | if (received_replies_[i].context->TryToUnblockListener(message)) { |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 163 | delete message; |
| 164 | received_replies_.erase(received_replies_.begin() + i); |
| 165 | return; |
| 166 | } |
| 167 | } |
| 168 | } |
| 169 | |
[email protected] | ac0efda | 2009-10-14 16:22:02 | [diff] [blame] | 170 | base::WaitableEventWatcher* top_send_done_watcher() { |
| 171 | return top_send_done_watcher_; |
| 172 | } |
| 173 | |
| 174 | void set_top_send_done_watcher(base::WaitableEventWatcher* watcher) { |
| 175 | top_send_done_watcher_ = watcher; |
| 176 | } |
| 177 | |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 178 | private: |
[email protected] | 877d55d | 2009-11-05 21:53:08 | [diff] [blame] | 179 | friend class base::RefCountedThreadSafe<ReceivedSyncMsgQueue>; |
| 180 | |
[email protected] | 4df10d61 | 2008-11-12 00:38:26 | [diff] [blame] | 181 | // See the comment in SyncChannel::SyncChannel for why this event is created |
| 182 | // as manual reset. |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 183 | ReceivedSyncMsgQueue() : |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 184 | message_queue_version_(0), |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 185 | dispatch_event_(true, false), |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 186 | listener_task_runner_(base::ThreadTaskRunnerHandle::Get()), |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 187 | task_pending_(false), |
[email protected] | ac0efda | 2009-10-14 16:22:02 | [diff] [blame] | 188 | listener_count_(0), |
| 189 | top_send_done_watcher_(NULL) { |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 190 | } |
| 191 | |
[email protected] | 877d55d | 2009-11-05 21:53:08 | [diff] [blame] | 192 | ~ReceivedSyncMsgQueue() {} |
| 193 | |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 194 | // Holds information about a queued synchronous message or reply. |
| 195 | struct QueuedMessage { |
| 196 | QueuedMessage(Message* m, SyncContext* c) : message(m), context(c) { } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 197 | Message* message; |
| 198 | scoped_refptr<SyncChannel::SyncContext> context; |
| 199 | }; |
| 200 | |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 201 | typedef std::list<QueuedMessage> SyncMessageQueue; |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 202 | SyncMessageQueue message_queue_; |
[email protected] | 522cc10d | 2012-01-11 22:39:54 | [diff] [blame] | 203 | uint32 message_queue_version_; // Used to signal DispatchMessages to rescan |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 204 | |
| 205 | std::vector<QueuedMessage> received_replies_; |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 206 | |
| 207 | // Set when we got a synchronous message that we must respond to as the |
| 208 | // sender needs its reply before it can reply to our original synchronous |
| 209 | // message. |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 210 | WaitableEvent dispatch_event_; |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 211 | scoped_refptr<base::SingleThreadTaskRunner> listener_task_runner_; |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 212 | base::Lock message_lock_; |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 213 | bool task_pending_; |
| 214 | int listener_count_; |
[email protected] | ac0efda | 2009-10-14 16:22:02 | [diff] [blame] | 215 | |
| 216 | // The current send done event watcher for this thread. Used to maintain |
| 217 | // a local global stack of send done watchers to ensure that nested sync |
| 218 | // message loops complete correctly. |
| 219 | base::WaitableEventWatcher* top_send_done_watcher_; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 220 | }; |
| 221 | |
[email protected] | f886b7bf | 2008-09-10 10:54:06 | [diff] [blame] | 222 | base::LazyInstance<base::ThreadLocalPointer<SyncChannel::ReceivedSyncMsgQueue> > |
[email protected] | 6de0fd1d | 2011-11-15 13:31:49 | [diff] [blame] | 223 | SyncChannel::ReceivedSyncMsgQueue::lazy_tls_ptr_ = |
| 224 | LAZY_INSTANCE_INITIALIZER; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 225 | |
| 226 | SyncChannel::SyncContext::SyncContext( |
[email protected] | b7f59e82 | 2012-06-29 22:05:26 | [diff] [blame] | 227 | Listener* listener, |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 228 | base::SingleThreadTaskRunner* ipc_task_runner, |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 229 | WaitableEvent* shutdown_event) |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 230 | : ChannelProxy::Context(listener, ipc_task_runner), |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 231 | received_sync_msgs_(ReceivedSyncMsgQueue::AddContext()), |
[email protected] | 54af05f | 2011-04-08 03:38:21 | [diff] [blame] | 232 | shutdown_event_(shutdown_event), |
[email protected] | 298ee7d | 2012-03-30 21:29:30 | [diff] [blame] | 233 | restrict_dispatch_group_(kRestrictDispatchGroup_None) { |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 234 | } |
| 235 | |
| 236 | SyncChannel::SyncContext::~SyncContext() { |
| 237 | while (!deserializers_.empty()) |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 238 | Pop(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 239 | } |
| 240 | |
| 241 | // Adds information about an outgoing sync message to the context so that |
| 242 | // we know how to deserialize the reply. Returns a handle that's set when |
| 243 | // the reply has arrived. |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 244 | void SyncChannel::SyncContext::Push(SyncMessage* sync_msg) { |
[email protected] | 4a180a5 | 2011-04-15 19:07:43 | [diff] [blame] | 245 | // Create the tracking information for this message. This object is stored |
| 246 | // by value since all members are pointers that are cheap to copy. These |
| 247 | // pointers are cleaned up in the Pop() function. |
| 248 | // |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 249 | // The event is created as manual reset because in between Signal and |
[email protected] | 4df10d61 | 2008-11-12 00:38:26 | [diff] [blame] | 250 | // OnObjectSignalled, another Send can happen which would stop the watcher |
| 251 | // from being called. The event would get watched later, when the nested |
| 252 | // Send completes, so the event will need to remain set. |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 253 | PendingSyncMsg pending(SyncMessage::GetMessageId(*sync_msg), |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 254 | sync_msg->GetReplyDeserializer(), |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 255 | new WaitableEvent(true, false)); |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 256 | base::AutoLock auto_lock(deserializers_lock_); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 257 | deserializers_.push_back(pending); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 258 | } |
| 259 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 260 | bool SyncChannel::SyncContext::Pop() { |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 261 | bool result; |
| 262 | { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 263 | base::AutoLock auto_lock(deserializers_lock_); |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 264 | PendingSyncMsg msg = deserializers_.back(); |
| 265 | delete msg.deserializer; |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 266 | delete msg.done_event; |
| 267 | msg.done_event = NULL; |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 268 | deserializers_.pop_back(); |
| 269 | result = msg.send_result; |
| 270 | } |
| 271 | |
| 272 | // We got a reply to a synchronous Send() call that's blocking the listener |
| 273 | // thread. However, further down the call stack there could be another |
| 274 | // blocking Send() call, whose reply we received after we made this last |
| 275 | // Send() call. So check if we have any queued replies available that |
| 276 | // can now unblock the listener thread. |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 277 | ipc_task_runner()->PostTask( |
[email protected] | 72b6f8e2 | 2011-11-12 21:16:41 | [diff] [blame] | 278 | FROM_HERE, base::Bind(&ReceivedSyncMsgQueue::DispatchReplies, |
| 279 | received_sync_msgs_.get())); |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 280 | |
| 281 | return result; |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 282 | } |
| 283 | |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 284 | WaitableEvent* SyncChannel::SyncContext::GetSendDoneEvent() { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 285 | base::AutoLock auto_lock(deserializers_lock_); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 286 | return deserializers_.back().done_event; |
| 287 | } |
| 288 | |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 289 | WaitableEvent* SyncChannel::SyncContext::GetDispatchEvent() { |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 290 | return received_sync_msgs_->dispatch_event(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 291 | } |
| 292 | |
| 293 | void SyncChannel::SyncContext::DispatchMessages() { |
[email protected] | 54af05f | 2011-04-08 03:38:21 | [diff] [blame] | 294 | received_sync_msgs_->DispatchMessages(this); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 295 | } |
| 296 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 297 | bool SyncChannel::SyncContext::TryToUnblockListener(const Message* msg) { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 298 | base::AutoLock auto_lock(deserializers_lock_); |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 299 | if (deserializers_.empty() || |
| 300 | !SyncMessage::IsMessageReplyTo(*msg, deserializers_.back().id)) { |
| 301 | return false; |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 302 | } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 303 | |
[email protected] | 211142cd | 2012-08-13 09:41:19 | [diff] [blame^] | 304 | // TODO(bauerb): Remove logging once investigation of http://crbug.com/141055 |
| 305 | // has finished. |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 306 | if (!msg->is_reply_error()) { |
[email protected] | 211142cd | 2012-08-13 09:41:19 | [diff] [blame^] | 307 | bool send_result = deserializers_.back().deserializer-> |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 308 | SerializeOutputParameters(*msg); |
[email protected] | 211142cd | 2012-08-13 09:41:19 | [diff] [blame^] | 309 | deserializers_.back().send_result = send_result; |
| 310 | LOG_IF(ERROR, !send_result) << "Couldn't deserialize reply message"; |
| 311 | } else { |
| 312 | LOG(ERROR) << "Received error reply"; |
[email protected] | 63a7bb8 | 2008-10-25 00:46:00 | [diff] [blame] | 313 | } |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 314 | deserializers_.back().done_event->Signal(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 315 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 316 | return true; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 317 | } |
| 318 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 319 | void SyncChannel::SyncContext::Clear() { |
| 320 | CancelPendingSends(); |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 321 | received_sync_msgs_->RemoveContext(this); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 322 | Context::Clear(); |
| 323 | } |
| 324 | |
[email protected] | a95986a8 | 2010-12-24 06:19:28 | [diff] [blame] | 325 | bool SyncChannel::SyncContext::OnMessageReceived(const Message& msg) { |
[email protected] | d65cab7a | 2008-08-12 01:25:41 | [diff] [blame] | 326 | // Give the filters a chance at processing this message. |
| 327 | if (TryFilters(msg)) |
[email protected] | a95986a8 | 2010-12-24 06:19:28 | [diff] [blame] | 328 | return true; |
[email protected] | d65cab7a | 2008-08-12 01:25:41 | [diff] [blame] | 329 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 330 | if (TryToUnblockListener(&msg)) |
[email protected] | a95986a8 | 2010-12-24 06:19:28 | [diff] [blame] | 331 | return true; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 332 | |
[email protected] | 9134cce6d | 2012-04-10 20:07:53 | [diff] [blame] | 333 | if (msg.is_reply()) { |
| 334 | received_sync_msgs_->QueueReply(msg, this); |
[email protected] | a95986a8 | 2010-12-24 06:19:28 | [diff] [blame] | 335 | return true; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 336 | } |
| 337 | |
[email protected] | 9134cce6d | 2012-04-10 20:07:53 | [diff] [blame] | 338 | if (msg.should_unblock()) { |
| 339 | received_sync_msgs_->QueueMessage(msg, this); |
[email protected] | a95986a8 | 2010-12-24 06:19:28 | [diff] [blame] | 340 | return true; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 341 | } |
| 342 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 343 | return Context::OnMessageReceivedNoFilter(msg); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 344 | } |
| 345 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 346 | void SyncChannel::SyncContext::OnChannelError() { |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 347 | CancelPendingSends(); |
[email protected] | a4f82270 | 2009-02-06 00:44:53 | [diff] [blame] | 348 | shutdown_watcher_.StopWatching(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 349 | Context::OnChannelError(); |
| 350 | } |
| 351 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 352 | void SyncChannel::SyncContext::OnChannelOpened() { |
| 353 | shutdown_watcher_.StartWatching(shutdown_event_, this); |
| 354 | Context::OnChannelOpened(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 355 | } |
| 356 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 357 | void SyncChannel::SyncContext::OnChannelClosed() { |
[email protected] | 87339f0 | 2010-09-02 21:45:50 | [diff] [blame] | 358 | CancelPendingSends(); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 359 | shutdown_watcher_.StopWatching(); |
| 360 | Context::OnChannelClosed(); |
| 361 | } |
| 362 | |
| 363 | void SyncChannel::SyncContext::OnSendTimeout(int message_id) { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 364 | base::AutoLock auto_lock(deserializers_lock_); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 365 | PendingSyncMessageQueue::iterator iter; |
[email protected] | 211142cd | 2012-08-13 09:41:19 | [diff] [blame^] | 366 | LOG(ERROR) << "Send timeout"; |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 367 | for (iter = deserializers_.begin(); iter != deserializers_.end(); iter++) { |
[email protected] | d3ae7a07 | 2008-12-05 20:27:20 | [diff] [blame] | 368 | if (iter->id == message_id) { |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 369 | iter->done_event->Signal(); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 370 | break; |
| 371 | } |
| 372 | } |
| 373 | } |
| 374 | |
| 375 | void SyncChannel::SyncContext::CancelPendingSends() { |
[email protected] | 20305ec | 2011-01-21 04:55:52 | [diff] [blame] | 376 | base::AutoLock auto_lock(deserializers_lock_); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 377 | PendingSyncMessageQueue::iterator iter; |
[email protected] | 211142cd | 2012-08-13 09:41:19 | [diff] [blame^] | 378 | LOG(ERROR) << "Canceling pending sends"; |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 379 | for (iter = deserializers_.begin(); iter != deserializers_.end(); iter++) |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 380 | iter->done_event->Signal(); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 381 | } |
| 382 | |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 383 | void SyncChannel::SyncContext::OnWaitableEventSignaled(WaitableEvent* event) { |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 384 | if (event == shutdown_event_) { |
| 385 | // Process shut down before we can get a reply to a synchronous message. |
| 386 | // Cancel pending Send calls, which will end up setting the send done event. |
| 387 | CancelPendingSends(); |
| 388 | } else { |
| 389 | // We got the reply, timed out or the process shutdown. |
[email protected] | 5e0be64 | 2011-04-28 18:20:09 | [diff] [blame] | 390 | DCHECK_EQ(GetSendDoneEvent(), event); |
[email protected] | 781a7ed | 2010-02-23 07:12:22 | [diff] [blame] | 391 | MessageLoop::current()->QuitNow(); |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 392 | } |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 393 | } |
| 394 | |
| 395 | |
| 396 | SyncChannel::SyncChannel( |
[email protected] | 42ce94e | 2010-12-08 19:28:09 | [diff] [blame] | 397 | const IPC::ChannelHandle& channel_handle, |
[email protected] | 4b580bf | 2010-12-02 19:16:07 | [diff] [blame] | 398 | Channel::Mode mode, |
[email protected] | b7f59e82 | 2012-06-29 22:05:26 | [diff] [blame] | 399 | Listener* listener, |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 400 | base::SingleThreadTaskRunner* ipc_task_runner, |
[email protected] | 4b580bf | 2010-12-02 19:16:07 | [diff] [blame] | 401 | bool create_pipe_now, |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 402 | WaitableEvent* shutdown_event) |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 403 | : ChannelProxy(new SyncContext(listener, ipc_task_runner, shutdown_event)), |
[email protected] | d65cab7a | 2008-08-12 01:25:41 | [diff] [blame] | 404 | sync_messages_with_no_timeout_allowed_(true) { |
[email protected] | 952394af | 2011-11-16 01:06:46 | [diff] [blame] | 405 | ChannelProxy::Init(channel_handle, mode, create_pipe_now); |
| 406 | StartWatching(); |
| 407 | } |
| 408 | |
| 409 | SyncChannel::SyncChannel( |
[email protected] | b7f59e82 | 2012-06-29 22:05:26 | [diff] [blame] | 410 | Listener* listener, |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 411 | base::SingleThreadTaskRunner* ipc_task_runner, |
[email protected] | 952394af | 2011-11-16 01:06:46 | [diff] [blame] | 412 | WaitableEvent* shutdown_event) |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 413 | : ChannelProxy(new SyncContext(listener, ipc_task_runner, shutdown_event)), |
[email protected] | 952394af | 2011-11-16 01:06:46 | [diff] [blame] | 414 | sync_messages_with_no_timeout_allowed_(true) { |
| 415 | StartWatching(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 416 | } |
| 417 | |
| 418 | SyncChannel::~SyncChannel() { |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 419 | } |
| 420 | |
[email protected] | 298ee7d | 2012-03-30 21:29:30 | [diff] [blame] | 421 | void SyncChannel::SetRestrictDispatchChannelGroup(int group) { |
| 422 | sync_context()->set_restrict_dispatch_group(group); |
[email protected] | 54af05f | 2011-04-08 03:38:21 | [diff] [blame] | 423 | } |
| 424 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 425 | bool SyncChannel::Send(Message* message) { |
[email protected] | aa96ae77 | 2009-01-20 22:08:15 | [diff] [blame] | 426 | return SendWithTimeout(message, base::kNoTimeout); |
[email protected] | d65cab7a | 2008-08-12 01:25:41 | [diff] [blame] | 427 | } |
| 428 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 429 | bool SyncChannel::SendWithTimeout(Message* message, int timeout_ms) { |
| 430 | if (!message->is_sync()) { |
| 431 | ChannelProxy::Send(message); |
| 432 | return true; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 433 | } |
| 434 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 435 | // *this* might get deleted in WaitForReply. |
| 436 | scoped_refptr<SyncContext> context(sync_context()); |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 437 | if (context->shutdown_event()->IsSignaled()) { |
[email protected] | 211142cd | 2012-08-13 09:41:19 | [diff] [blame^] | 438 | LOG(ERROR) << "shutdown event is signaled"; |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 439 | delete message; |
| 440 | return false; |
| 441 | } |
| 442 | |
[email protected] | d321644 | 2009-03-05 21:07:27 | [diff] [blame] | 443 | DCHECK(sync_messages_with_no_timeout_allowed_ || |
| 444 | timeout_ms != base::kNoTimeout); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 445 | SyncMessage* sync_msg = static_cast<SyncMessage*>(message); |
| 446 | context->Push(sync_msg); |
| 447 | int message_id = SyncMessage::GetMessageId(*sync_msg); |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 448 | WaitableEvent* pump_messages_event = sync_msg->pump_messages_event(); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 449 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 450 | ChannelProxy::Send(message); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 451 | |
[email protected] | aa96ae77 | 2009-01-20 22:08:15 | [diff] [blame] | 452 | if (timeout_ms != base::kNoTimeout) { |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 453 | // We use the sync message id so that when a message times out, we don't |
| 454 | // confuse it with another send that is either above/below this Send in |
| 455 | // the call stack. |
[email protected] | b243230 | 2012-07-02 21:15:52 | [diff] [blame] | 456 | context->ipc_task_runner()->PostDelayedTask( |
[email protected] | 72b6f8e2 | 2011-11-12 21:16:41 | [diff] [blame] | 457 | FROM_HERE, |
| 458 | base::Bind(&SyncContext::OnSendTimeout, context.get(), message_id), |
[email protected] | 5896fa04 | 2012-03-07 04:41:40 | [diff] [blame] | 459 | base::TimeDelta::FromMilliseconds(timeout_ms)); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 460 | } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 461 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 462 | // Wait for reply, or for any other incoming synchronous messages. |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 463 | // *this* might get deleted, so only call static functions at this point. |
| 464 | WaitForReply(context, pump_messages_event); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 465 | |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 466 | return context->Pop(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 467 | } |
| 468 | |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 469 | void SyncChannel::WaitForReply( |
| 470 | SyncContext* context, WaitableEvent* pump_messages_event) { |
[email protected] | 54af05f | 2011-04-08 03:38:21 | [diff] [blame] | 471 | context->DispatchMessages(); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 472 | while (true) { |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 473 | WaitableEvent* objects[] = { |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 474 | context->GetDispatchEvent(), |
| 475 | context->GetSendDoneEvent(), |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 476 | pump_messages_event |
| 477 | }; |
| 478 | |
| 479 | unsigned count = pump_messages_event ? 3: 2; |
[email protected] | 7dc8d79 | 2009-11-20 17:30:44 | [diff] [blame] | 480 | size_t result = WaitableEvent::WaitMany(objects, count); |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 481 | if (result == 0 /* dispatch event */) { |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 482 | // We're waiting for a reply, but we received a blocking synchronous |
| 483 | // call. We must process it or otherwise a deadlock might occur. |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 484 | context->GetDispatchEvent()->Reset(); |
| 485 | context->DispatchMessages(); |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 486 | continue; |
| 487 | } |
| 488 | |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 489 | if (result == 2 /* pump_messages_event */) |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 490 | WaitForReplyWithNestedMessageLoop(context); // Run a nested message loop. |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 491 | |
| 492 | break; |
| 493 | } |
| 494 | } |
| 495 | |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 496 | void SyncChannel::WaitForReplyWithNestedMessageLoop(SyncContext* context) { |
[email protected] | ac0efda | 2009-10-14 16:22:02 | [diff] [blame] | 497 | base::WaitableEventWatcher send_done_watcher; |
| 498 | |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 499 | ReceivedSyncMsgQueue* sync_msg_queue = context->received_sync_msgs(); |
[email protected] | ac0efda | 2009-10-14 16:22:02 | [diff] [blame] | 500 | DCHECK(sync_msg_queue != NULL); |
| 501 | |
| 502 | base::WaitableEventWatcher* old_send_done_event_watcher = |
| 503 | sync_msg_queue->top_send_done_watcher(); |
| 504 | |
| 505 | base::WaitableEventWatcher::Delegate* old_delegate = NULL; |
| 506 | base::WaitableEvent* old_event = NULL; |
| 507 | |
| 508 | // Maintain a local global stack of send done delegates to ensure that |
| 509 | // nested sync calls complete in the correct sequence, i.e. the |
| 510 | // outermost call completes first, etc. |
| 511 | if (old_send_done_event_watcher) { |
| 512 | old_delegate = old_send_done_event_watcher->delegate(); |
| 513 | old_event = old_send_done_event_watcher->GetWatchedEvent(); |
| 514 | old_send_done_event_watcher->StopWatching(); |
| 515 | } |
| 516 | |
| 517 | sync_msg_queue->set_top_send_done_watcher(&send_done_watcher); |
| 518 | |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 519 | send_done_watcher.StartWatching(context->GetSendDoneEvent(), context); |
[email protected] | ac0efda | 2009-10-14 16:22:02 | [diff] [blame] | 520 | |
[email protected] | b5717a4 | 2012-02-14 19:33:52 | [diff] [blame] | 521 | { |
| 522 | MessageLoop::ScopedNestableTaskAllower allow(MessageLoop::current()); |
| 523 | MessageLoop::current()->Run(); |
| 524 | } |
[email protected] | ac0efda | 2009-10-14 16:22:02 | [diff] [blame] | 525 | |
| 526 | sync_msg_queue->set_top_send_done_watcher(old_send_done_event_watcher); |
[email protected] | 42437980 | 2009-10-14 19:58:13 | [diff] [blame] | 527 | if (old_send_done_event_watcher && old_event) { |
[email protected] | ac0efda | 2009-10-14 16:22:02 | [diff] [blame] | 528 | old_send_done_event_watcher->StartWatching(old_event, old_delegate); |
| 529 | } |
[email protected] | 3cdb7af81 | 2008-10-24 19:21:13 | [diff] [blame] | 530 | } |
| 531 | |
[email protected] | 1c4947f | 2009-01-15 22:25:11 | [diff] [blame] | 532 | void SyncChannel::OnWaitableEventSignaled(WaitableEvent* event) { |
[email protected] | 9eec225 | 2009-12-01 02:34:18 | [diff] [blame] | 533 | DCHECK(event == sync_context()->GetDispatchEvent()); |
| 534 | // The call to DispatchMessages might delete this object, so reregister |
| 535 | // the object watcher first. |
| 536 | event->Reset(); |
| 537 | dispatch_watcher_.StartWatching(event, this); |
| 538 | sync_context()->DispatchMessages(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 539 | } |
| 540 | |
[email protected] | 952394af | 2011-11-16 01:06:46 | [diff] [blame] | 541 | void SyncChannel::StartWatching() { |
| 542 | // Ideally we only want to watch this object when running a nested message |
| 543 | // loop. However, we don't know when it exits if there's another nested |
| 544 | // message loop running under it or not, so we wouldn't know whether to |
| 545 | // stop or keep watching. So we always watch it, and create the event as |
| 546 | // manual reset since the object watcher might otherwise reset the event |
| 547 | // when we're doing a WaitMany. |
| 548 | dispatch_watcher_.StartWatching(sync_context()->GetDispatchEvent(), this); |
| 549 | } |
| 550 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 551 | } // namespace IPC |