OLD | NEW |
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/message_pump/handle_watcher.h" | 5 #include "mojo/message_pump/handle_watcher.h" |
6 | 6 |
7 #include <map> | 7 #include <map> |
8 | 8 |
9 #include "base/atomic_sequence_num.h" | 9 #include "base/atomic_sequence_num.h" |
10 #include "base/bind.h" | 10 #include "base/bind.h" |
(...skipping 204 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
215 Requests requests_; | 215 Requests requests_; |
216 | 216 |
217 DISALLOW_COPY_AND_ASSIGN(WatcherThreadManager); | 217 DISALLOW_COPY_AND_ASSIGN(WatcherThreadManager); |
218 }; | 218 }; |
219 | 219 |
220 WatcherThreadManager::~WatcherThreadManager() { | 220 WatcherThreadManager::~WatcherThreadManager() { |
221 thread_.Stop(); | 221 thread_.Stop(); |
222 } | 222 } |
223 | 223 |
224 WatcherThreadManager* WatcherThreadManager::GetInstance() { | 224 WatcherThreadManager* WatcherThreadManager::GetInstance() { |
225 return base::Singleton<WatcherThreadManager>::get(); | 225 // We need to leak this because otherwise when the process dies, AtExitManager |
| 226 // waits for destruction which waits till the handle watcher thread is joined. |
| 227 // But that can't happen since the pump uses mojo message pipes to wake up the |
| 228 // pump. Since mojo EDK has been shutdown already, this never completes. |
| 229 return base::Singleton<WatcherThreadManager, |
| 230 base::LeakySingletonTraits<WatcherThreadManager>>:: |
| 231 get(); |
226 } | 232 } |
227 | 233 |
228 WatcherID WatcherThreadManager::StartWatching( | 234 WatcherID WatcherThreadManager::StartWatching( |
229 const Handle& handle, | 235 const Handle& handle, |
230 MojoHandleSignals handle_signals, | 236 MojoHandleSignals handle_signals, |
231 base::TimeTicks deadline, | 237 base::TimeTicks deadline, |
232 const base::Callback<void(MojoResult)>& callback) { | 238 const base::Callback<void(MojoResult)>& callback) { |
233 RequestData request_data; | 239 RequestData request_data; |
234 request_data.type = REQUEST_START; | 240 request_data.type = REQUEST_START; |
235 request_data.start_data.id = watcher_id_generator_.GetNext(); | 241 request_data.start_data.id = watcher_id_generator_.GetNext(); |
(...skipping 230 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
466 this, handle, handle_signals, deadline, callback)); | 472 this, handle, handle_signals, deadline, callback)); |
467 } | 473 } |
468 } | 474 } |
469 | 475 |
470 void HandleWatcher::Stop() { | 476 void HandleWatcher::Stop() { |
471 state_.reset(); | 477 state_.reset(); |
472 } | 478 } |
473 | 479 |
474 } // namespace common | 480 } // namespace common |
475 } // namespace mojo | 481 } // namespace mojo |
OLD | NEW |