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

Side by Side Diff: remoting/host/chromoting_host_context.cc

Issue 9582031: Don't crash in ~HostNPScriptObject() when the object wasn't initialized. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: - Created 8 years, 9 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
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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 #include "remoting/host/chromoting_host_context.h" 5 #include "remoting/host/chromoting_host_context.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/threading/thread.h" 10 #include "base/threading/thread.h"
11 #include "remoting/jingle_glue/jingle_thread.h" 11 #include "remoting/jingle_glue/jingle_thread.h"
12 12
13 namespace remoting { 13 namespace remoting {
14 14
15 ChromotingHostContext::ChromotingHostContext( 15 ChromotingHostContext::ChromotingHostContext(
16 base::MessageLoopProxy* ui_message_loop) 16 base::MessageLoopProxy* ui_message_loop)
17 : main_thread_("ChromotingMainThread"), 17 : main_thread_("ChromotingMainThread"),
18 encode_thread_("ChromotingEncodeThread"), 18 encode_thread_("ChromotingEncodeThread"),
19 desktop_thread_("ChromotingDesktopThread"), 19 desktop_thread_("ChromotingDesktopThread"),
20 ui_message_loop_(ui_message_loop) { 20 ui_message_loop_(ui_message_loop) {
21 } 21 }
22 22
23 ChromotingHostContext::~ChromotingHostContext() { 23 ChromotingHostContext::~ChromotingHostContext() {
24 } 24 }
25 25
26 void ChromotingHostContext::Start() { 26 bool ChromotingHostContext::Start() {
27 // Start all the threads. 27 // Start all the threads.
28 main_thread_.Start(); 28 return main_thread_.Start() && encode_thread_.Start() &&
29 encode_thread_.Start(); 29 jingle_thread_.Start() && desktop_thread_.Start();
30 jingle_thread_.Start();
31 desktop_thread_.Start();
32 }
33
34 void ChromotingHostContext::Stop() {
35 // Stop all the threads.
36 jingle_thread_.Stop();
37 encode_thread_.Stop();
38 main_thread_.Stop();
39 desktop_thread_.Stop();
40 } 30 }
41 31
42 JingleThread* ChromotingHostContext::jingle_thread() { 32 JingleThread* ChromotingHostContext::jingle_thread() {
43 return &jingle_thread_; 33 return &jingle_thread_;
44 } 34 }
45 35
46 base::MessageLoopProxy* ChromotingHostContext::ui_message_loop() { 36 base::MessageLoopProxy* ChromotingHostContext::ui_message_loop() {
47 return ui_message_loop_; 37 return ui_message_loop_;
48 } 38 }
49 39
50 MessageLoop* ChromotingHostContext::main_message_loop() { 40 MessageLoop* ChromotingHostContext::main_message_loop() {
51 return main_thread_.message_loop(); 41 return main_thread_.message_loop();
52 } 42 }
53 43
54 MessageLoop* ChromotingHostContext::encode_message_loop() { 44 MessageLoop* ChromotingHostContext::encode_message_loop() {
55 return encode_thread_.message_loop(); 45 return encode_thread_.message_loop();
56 } 46 }
57 47
58 base::MessageLoopProxy* ChromotingHostContext::network_message_loop() { 48 base::MessageLoopProxy* ChromotingHostContext::network_message_loop() {
59 return jingle_thread_.message_loop_proxy(); 49 return jingle_thread_.message_loop_proxy();
60 } 50 }
61 51
62 MessageLoop* ChromotingHostContext::desktop_message_loop() { 52 MessageLoop* ChromotingHostContext::desktop_message_loop() {
63 return desktop_thread_.message_loop(); 53 return desktop_thread_.message_loop();
64 } 54 }
65 55
66 } // namespace remoting 56 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/host/chromoting_host_context.h ('k') | remoting/host/chromoting_host_context_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698