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

Side by Side Diff: mojo/public/cpp/application/application_runner_chromium.h

Issue 506353002: Make HandleWatcher watch on the same thread if the thread is running a MessagePumpMojo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 MOJO_PUBLIC_APPLICATION_APPLICATION_RUNNER_CHROMIUM_H_ 5 #ifndef MOJO_PUBLIC_APPLICATION_APPLICATION_RUNNER_CHROMIUM_H_
6 #define MOJO_PUBLIC_APPLICATION_APPLICATION_RUNNER_CHROMIUM_H_ 6 #define MOJO_PUBLIC_APPLICATION_APPLICATION_RUNNER_CHROMIUM_H_
7 7
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "mojo/public/cpp/system/core.h" 10 #include "mojo/public/cpp/system/core.h"
11 11
12 namespace mojo { 12 namespace mojo {
13 13
14 class ApplicationDelegate; 14 class ApplicationDelegate;
15 15
16 // A utility for running a chromium based mojo Application. The typical use 16 // A utility for running a chromium based mojo Application. The typical use
17 // case is to use when writing your MojoMain: 17 // case is to use when writing your MojoMain:
18 // 18 //
19 // MojoResult MojoMain(MojoHandle shell_handle) { 19 // MojoResult MojoMain(MojoHandle shell_handle) {
20 // mojo::ApplicationRunnerChromium runner(new MyDelegate()); 20 // mojo::ApplicationRunnerChromium runner(new MyDelegate());
21 // return runner.Run(shell_handle); 21 // return runner.Run(shell_handle);
22 // } 22 // }
23 // 23 //
24 // ApplicationRunnerChromium takes care of chromium environment initialization 24 // ApplicationRunnerChromium takes care of chromium environment initialization
25 // and shutdown, and starting a RunLoop from which your application can run and 25 // and shutdown, and starting a MessageLoop from which your application can run
26 // ultimately Quit(). 26 // and ultimately Quit().
27 class ApplicationRunnerChromium { 27 class ApplicationRunnerChromium {
28 public: 28 public:
29 // Takes ownership of |delegate|. 29 // Takes ownership of |delegate|.
30 explicit ApplicationRunnerChromium(ApplicationDelegate* delegate); 30 explicit ApplicationRunnerChromium(ApplicationDelegate* delegate);
31 ~ApplicationRunnerChromium(); 31 ~ApplicationRunnerChromium();
32 32
33 void set_message_loop_type(base::MessageLoop::Type type); 33 void set_message_loop_type(base::MessageLoop::Type type);
34 34
35 // Once the various parameters have been set above, use Run to initialize an 35 // Once the various parameters have been set above, use Run to initialize an
36 // ApplicationImpl wired to the provided delegate, and run a RunLoop until 36 // ApplicationImpl wired to the provided delegate, and run a MessageLoop until
37 // the application exits. 37 // the application exits.
38 MojoResult Run(MojoHandle shell_handle); 38 MojoResult Run(MojoHandle shell_handle);
39 39
40 private: 40 private:
41 scoped_ptr<ApplicationDelegate> delegate_; 41 scoped_ptr<ApplicationDelegate> delegate_;
42 42
43 // MessageLoop type. TYPE_DEFAULT is default. 43 // MessageLoop type. TYPE_CUSTOM is default (MessagePumpMojo will be used as
44 // the underlying message pump).
44 base::MessageLoop::Type message_loop_type_; 45 base::MessageLoop::Type message_loop_type_;
46 // Whether Run() has been called.
47 bool has_run_;
45 48
46 MOJO_DISALLOW_COPY_AND_ASSIGN(ApplicationRunnerChromium); 49 MOJO_DISALLOW_COPY_AND_ASSIGN(ApplicationRunnerChromium);
47 }; 50 };
48 51
49 } // namespace mojo 52 } // namespace mojo
50 53
51 #endif // MOJO_PUBLIC_APPLICATION_APPLICATION_RUNNER_CHROMIUM_H_ 54 #endif // MOJO_PUBLIC_APPLICATION_APPLICATION_RUNNER_CHROMIUM_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698