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

Side by Side Diff: content/renderer/mojo/mojo_application.h

Issue 236813002: Move Mojo channel initialization closer to IPC::Channel setup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixup Created 6 years, 8 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #ifndef CONTENT_RENDERER_MOJO_MOJO_APPLICATION_H_
6 #define CONTENT_RENDERER_MOJO_MOJO_APPLICATION_H_
7
8 #include "content/common/mojo/mojo_channel_init.h"
9 #include "ipc/ipc_platform_file.h"
10 #include "mojo/public/cpp/bindings/remote_ptr.h"
11 #include "mojo/public/interfaces/shell/shell.mojom.h"
12
13 namespace IPC {
14 class Message;
15 }
16
17 namespace content {
18
19 class MojoApplication {
sky 2014/04/16 17:18:10 Description?
darin (slow to review) 2014/04/16 23:26:04 Will add.
20 public:
21 explicit MojoApplication(mojo::ShellClient* shell_client);
22 ~MojoApplication();
23
24 bool OnMessageReceived(const IPC::Message& msg);
25
26 mojo::Shell* shell() { return shell_.get(); }
27
28 private:
29 void OnActivate(const IPC::PlatformFileForTransit& file);
30
31 MojoChannelInit channel_init_;
32 mojo::RemotePtr<mojo::Shell> shell_;
33 mojo::ShellClient* shell_client_;
sky 2014/04/16 17:18:10 Document ownership.
darin (slow to review) 2014/04/16 23:26:04 Good idea.
34 };
sky 2014/04/16 17:18:10 DISALLOW...
darin (slow to review) 2014/04/16 23:26:04 Thanks.
35
36 } // namespace content
37
38 #endif // CONTENT_RENDERER_MOJO_MOJO_APPLICATION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698