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

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

Issue 11366226: Rename capture and encode threads to make it clear that they are for video. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month 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 | « remoting/host/chromoting_host.cc ('k') | remoting/host/chromoting_host_context.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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_ 5 #ifndef REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_
6 #define REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_ 6 #define REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_
7 7
8 #include "base/gtest_prod_util.h" 8 #include "base/gtest_prod_util.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "base/threading/thread.h" 10 #include "base/threading/thread.h"
(...skipping 26 matching lines...) Expand all
37 // should be able to remove the need for virtual functions below 37 // should be able to remove the need for virtual functions below
38 // with that design, while preserving the relative simplicity of 38 // with that design, while preserving the relative simplicity of
39 // this API. 39 // this API.
40 virtual bool Start(); 40 virtual bool Start();
41 41
42 // Task runner for the thread used for audio capture and encoding. 42 // Task runner for the thread used for audio capture and encoding.
43 virtual base::SingleThreadTaskRunner* audio_task_runner(); 43 virtual base::SingleThreadTaskRunner* audio_task_runner();
44 44
45 // Task runner for the thread used by the ScreenRecorder to capture 45 // Task runner for the thread used by the ScreenRecorder to capture
46 // the screen. 46 // the screen.
47 virtual base::SingleThreadTaskRunner* capture_task_runner(); 47 virtual base::SingleThreadTaskRunner* video_capture_task_runner();
48 48
49 // Task runner for the thread used to encode video streams. 49 // Task runner for the thread used to encode video streams.
50 virtual base::SingleThreadTaskRunner* encode_task_runner(); 50 virtual base::SingleThreadTaskRunner* video_encode_task_runner();
51 51
52 // Task runner for the thread that is used for blocking file 52 // Task runner for the thread that is used for blocking file
53 // IO. This thread is used by the URLRequestContext to read proxy 53 // IO. This thread is used by the URLRequestContext to read proxy
54 // configuration and by NatConfig to read policy configs. 54 // configuration and by NatConfig to read policy configs.
55 virtual base::SingleThreadTaskRunner* file_task_runner(); 55 virtual base::SingleThreadTaskRunner* file_task_runner();
56 56
57 // Task runner for the thread that is used by the EventExecutor. 57 // Task runner for the thread that is used by the EventExecutor.
58 // 58 //
59 // TODO(sergeyu): Do we need a separate thread for EventExecutor? 59 // TODO(sergeyu): Do we need a separate thread for EventExecutor?
60 // Can we use some other thread instead? 60 // Can we use some other thread instead?
(...skipping 11 matching lines...) Expand all
72 const scoped_refptr<net::URLRequestContextGetter>& 72 const scoped_refptr<net::URLRequestContextGetter>&
73 url_request_context_getter(); 73 url_request_context_getter();
74 74
75 private: 75 private:
76 FRIEND_TEST_ALL_PREFIXES(ChromotingHostContextTest, StartAndStop); 76 FRIEND_TEST_ALL_PREFIXES(ChromotingHostContextTest, StartAndStop);
77 77
78 // A thread that hosts audio capture and encoding. 78 // A thread that hosts audio capture and encoding.
79 base::Thread audio_thread_; 79 base::Thread audio_thread_;
80 80
81 // A thread that hosts screen capture. 81 // A thread that hosts screen capture.
82 base::Thread capture_thread_; 82 base::Thread video_capture_thread_;
83 83
84 // A thread that hosts all encode operations. 84 // A thread that hosts video encode operations.
85 base::Thread encode_thread_; 85 base::Thread video_encode_thread_;
86 86
87 // Thread for blocking IO operations. 87 // Thread for blocking IO operations.
88 base::Thread file_thread_; 88 base::Thread file_thread_;
89 89
90 // A thread that hosts input injection. 90 // A thread that hosts input injection.
91 base::Thread input_thread_; 91 base::Thread input_thread_;
92 92
93 // A thread that hosts all network operations. 93 // A thread that hosts all network operations.
94 base::Thread network_thread_; 94 base::Thread network_thread_;
95 95
96 // Task runners wrapping the above threads. These should be declared after 96 // Task runners wrapping the above threads. These should be declared after
97 // the corresponding threads to guarantee proper order of destruction. 97 // the corresponding threads to guarantee proper order of destruction.
98 scoped_refptr<base::SingleThreadTaskRunner> audio_task_runner_; 98 scoped_refptr<base::SingleThreadTaskRunner> audio_task_runner_;
99 scoped_refptr<base::SingleThreadTaskRunner> capture_task_runner_; 99 scoped_refptr<base::SingleThreadTaskRunner> video_capture_task_runner_;
100 scoped_refptr<base::SingleThreadTaskRunner> encode_task_runner_; 100 scoped_refptr<base::SingleThreadTaskRunner> video_encode_task_runner_;
101 scoped_refptr<base::SingleThreadTaskRunner> file_task_runner_; 101 scoped_refptr<base::SingleThreadTaskRunner> file_task_runner_;
102 scoped_refptr<base::SingleThreadTaskRunner> input_task_runner_; 102 scoped_refptr<base::SingleThreadTaskRunner> input_task_runner_;
103 scoped_refptr<base::SingleThreadTaskRunner> network_task_runner_; 103 scoped_refptr<base::SingleThreadTaskRunner> network_task_runner_;
104 104
105 scoped_refptr<AutoThreadTaskRunner> ui_task_runner_; 105 scoped_refptr<AutoThreadTaskRunner> ui_task_runner_;
106 106
107 scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_; 107 scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_;
108 108
109 DISALLOW_COPY_AND_ASSIGN(ChromotingHostContext); 109 DISALLOW_COPY_AND_ASSIGN(ChromotingHostContext);
110 }; 110 };
111 111
112 } // namespace remoting 112 } // namespace remoting
113 113
114 #endif // REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_ 114 #endif // REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_
OLDNEW
« no previous file with comments | « remoting/host/chromoting_host.cc ('k') | remoting/host/chromoting_host_context.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698