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

Side by Side Diff: base/threading/thread.h

Issue 1011683002: Lazily initialize MessageLoop for faster thread startup (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
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 BASE_THREADING_THREAD_H_ 5 #ifndef BASE_THREADING_THREAD_H_
6 #define BASE_THREADING_THREAD_H_ 6 #define BASE_THREADING_THREAD_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/base_export.h" 10 #include "base/base_export.h"
11 #include "base/callback.h" 11 #include "base/callback.h"
12 #include "base/memory/scoped_ptr.h" 12 #include "base/memory/scoped_ptr.h"
13 #include "base/message_loop/message_loop.h" 13 #include "base/message_loop/message_loop.h"
14 #include "base/message_loop/message_loop_proxy.h" 14 #include "base/message_loop/message_loop_proxy.h"
15 #include "base/message_loop/timer_slack.h" 15 #include "base/message_loop/timer_slack.h"
16 #include "base/synchronization/lock.h"
16 #include "base/threading/platform_thread.h" 17 #include "base/threading/platform_thread.h"
17 18
18 namespace base { 19 namespace base {
19 20
20 class MessagePump; 21 class MessagePump;
22 class WaitableEvent;
21 23
22 // A simple thread abstraction that establishes a MessageLoop on a new thread. 24 // A simple thread abstraction that establishes a MessageLoop on a new thread.
23 // The consumer uses the MessageLoop of the thread to cause code to execute on 25 // The consumer uses the MessageLoop of the thread to cause code to execute on
24 // the thread. When this object is destroyed the thread is terminated. All 26 // the thread. When this object is destroyed the thread is terminated. All
25 // pending tasks queued on the thread's message loop will run to completion 27 // pending tasks queued on the thread's message loop will run to completion
26 // before the thread is terminated. 28 // before the thread is terminated.
27 // 29 //
28 // WARNING! SUBCLASSES MUST CALL Stop() IN THEIR DESTRUCTORS! See ~Thread(). 30 // WARNING! SUBCLASSES MUST CALL Stop() IN THEIR DESTRUCTORS! See ~Thread().
29 // 31 //
30 // After the thread is stopped, the destruction sequence is: 32 // After the thread is stopped, the destruction sequence is:
31 // 33 //
32 // (1) Thread::CleanUp() 34 // (1) Thread::CleanUp()
33 // (2) MessageLoop::~MessageLoop 35 // (2) MessageLoop::~MessageLoop
34 // (3.b) MessageLoop::DestructionObserver::WillDestroyCurrentMessageLoop 36 // (3.b) MessageLoop::DestructionObserver::WillDestroyCurrentMessageLoop
35 class BASE_EXPORT Thread : PlatformThread::Delegate { 37 class BASE_EXPORT Thread : PlatformThread::Delegate {
36 public: 38 public:
37 struct BASE_EXPORT Options { 39 struct BASE_EXPORT Options {
38 typedef Callback<scoped_ptr<MessagePump>()> MessagePumpFactory; 40 typedef Callback<scoped_ptr<MessagePump>()> MessagePumpFactory;
39 41
40 Options(); 42 Options();
41 Options(MessageLoop::Type type, size_t size); 43 Options(MessageLoop::Type type, size_t size);
42 ~Options(); 44 ~Options();
43 45
44 // Specifies the type of message loop that will be allocated on the thread. 46 // Specifies the type of message loop that will be allocated on the thread.
45 // This is ignored if message_pump_factory.is_null() is false. 47 // This is ignored if message_pump_factory.is_null() is false.
46 MessageLoop::Type message_loop_type; 48 MessageLoop::Type message_loop_type;
47 49
48 // Specify timer slack for thread message loop. 50 // Specifies timer slack for thread message loop.
49 TimerSlack timer_slack; 51 TimerSlack timer_slack;
50 52
51 // Used to create the MessagePump for the MessageLoop. The callback is Run() 53 // Used to create the MessagePump for the MessageLoop. The callback is Run()
52 // on the thread. If message_pump_factory.is_null(), then a MessagePump 54 // on the thread. If message_pump_factory.is_null(), then a MessagePump
53 // appropriate for |message_loop_type| is created. Setting this forces the 55 // appropriate for |message_loop_type| is created. Setting this forces the
54 // MessageLoop::Type to TYPE_CUSTOM. 56 // MessageLoop::Type to TYPE_CUSTOM.
55 MessagePumpFactory message_pump_factory; 57 MessagePumpFactory message_pump_factory;
56 58
57 // Specifies the maximum stack size that the thread is allowed to use. 59 // Specifies the maximum stack size that the thread is allowed to use.
58 // This does not necessarily correspond to the thread's initial stack size. 60 // This does not necessarily correspond to the thread's initial stack size.
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 bool Start(); 98 bool Start();
97 99
98 // Starts the thread. Behaves exactly like Start in addition to allow to 100 // Starts the thread. Behaves exactly like Start in addition to allow to
99 // override the default options. 101 // override the default options.
100 // 102 //
101 // Note: This function can't be called on Windows with the loader lock held; 103 // Note: This function can't be called on Windows with the loader lock held;
102 // i.e. during a DllMain, global object construction or destruction, atexit() 104 // i.e. during a DllMain, global object construction or destruction, atexit()
103 // callback. 105 // callback.
104 bool StartWithOptions(const Options& options); 106 bool StartWithOptions(const Options& options);
105 107
108 // Starts the thread and wait for the thread to start and run initialization
109 // before returning. It's same as calling Start() and then
110 // WaitUntilThreadStarted().
111 // Note that using this (instead of Start() or StartWithOptions() causes
112 // jank on the calling thread, and must be used carefully for production code.
113 bool StartAndWait();
114
115 // Blocks until the thread starts running. Supposed to be used for testing.
116 bool WaitUntilThreadStarted();
117
106 // Signals the thread to exit and returns once the thread has exited. After 118 // Signals the thread to exit and returns once the thread has exited. After
107 // this method returns, the Thread object is completely reset and may be used 119 // this method returns, the Thread object is completely reset and may be used
108 // as if it were newly constructed (i.e., Start may be called again). 120 // as if it were newly constructed (i.e., Start may be called again).
109 // 121 //
110 // Stop may be called multiple times and is simply ignored if the thread is 122 // Stop may be called multiple times and is simply ignored if the thread is
111 // already stopped. 123 // already stopped.
112 // 124 //
113 // NOTE: If you are a consumer of Thread, it is not necessary to call this 125 // NOTE: If you are a consumer of Thread, it is not necessary to call this
114 // before deleting your Thread objects, as the destructor will do it. 126 // before deleting your Thread objects, as the destructor will do it.
115 // IF YOU ARE A SUBCLASS OF Thread, YOU MUST CALL THIS IN YOUR DESTRUCTOR. 127 // IF YOU ARE A SUBCLASS OF Thread, YOU MUST CALL THIS IN YOUR DESTRUCTOR.
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
196 enum ComStatus { 208 enum ComStatus {
197 NONE, 209 NONE,
198 STA, 210 STA,
199 MTA, 211 MTA,
200 }; 212 };
201 #endif 213 #endif
202 214
203 // PlatformThread::Delegate methods: 215 // PlatformThread::Delegate methods:
204 void ThreadMain() override; 216 void ThreadMain() override;
205 217
218 // Returns true while inside of Run().
219 bool IsInRunLoop() const;
220
206 #if defined(OS_WIN) 221 #if defined(OS_WIN)
207 // Whether this thread needs to initialize COM, and if so, in what mode. 222 // Whether this thread needs to initialize COM, and if so, in what mode.
208 ComStatus com_status_; 223 ComStatus com_status_;
209 #endif 224 #endif
210 225
211 // Whether we successfully started the thread. 226 // Whether we successfully started the thread.
212 bool started_; 227 bool started_;
213 228
214 // If true, we're in the middle of stopping, and shouldn't access 229 // If true, we're in the middle of stopping, and shouldn't access
215 // |message_loop_|. It may non-NULL and invalid. 230 // |message_loop_|. It may non-NULL and invalid.
216 bool stopping_; 231 bool stopping_;
217 232
218 // True while inside of Run(). 233 // True while inside of Run().
234 mutable base::Lock running_lock_;
219 bool running_; 235 bool running_;
220 236
221 // Used to pass data to ThreadMain.
222 struct StartupData;
223 StartupData* startup_data_;
224
225 // The thread's handle. 237 // The thread's handle.
226 PlatformThreadHandle thread_; 238 PlatformThreadHandle thread_;
227 239
228 // The thread's message loop. Valid only while the thread is alive. Set 240 // The thread's message loop. Valid only while the thread is alive. Set
229 // by the created thread. 241 // by the created thread.
230 MessageLoop* message_loop_; 242 MessageLoop* message_loop_;
231 243
232 // Our thread's ID. 244 // Our thread's ID.
233 PlatformThreadId thread_id_; 245 PlatformThreadId thread_id_;
234 246
235 // The name of the thread. Used for debugging purposes. 247 // The name of the thread. Used for debugging purposes.
236 std::string name_; 248 std::string name_;
237 249
250 scoped_ptr<WaitableEvent> start_event_;
251
238 friend void ThreadQuitHelper(); 252 friend void ThreadQuitHelper();
239 253
240 DISALLOW_COPY_AND_ASSIGN(Thread); 254 DISALLOW_COPY_AND_ASSIGN(Thread);
241 }; 255 };
242 256
243 } // namespace base 257 } // namespace base
244 258
245 #endif // BASE_THREADING_THREAD_H_ 259 #endif // BASE_THREADING_THREAD_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698