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

Side by Side Diff: services/native_support/main.cc

Issue 1975993002: Change InterfaceFactory<I>::Create() to take a ConnectionContext instead of an ApplicationConnectio… (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 years, 7 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
« no previous file with comments | « services/nacl/nonsfi/pnacl_link.cc ('k') | services/native_support/process_impl.h » ('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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "base/macros.h" 5 #include "base/macros.h"
6 #include "base/memory/ref_counted.h" 6 #include "base/memory/ref_counted.h"
7 #include "base/message_loop/message_loop.h" 7 #include "base/message_loop/message_loop.h"
8 #include "base/threading/sequenced_worker_pool.h" 8 #include "base/threading/sequenced_worker_pool.h"
9 #include "mojo/application/application_runner_chromium.h" 9 #include "mojo/application/application_runner_chromium.h"
10 #include "mojo/public/c/system/main.h" 10 #include "mojo/public/c/system/main.h"
(...skipping 19 matching lines...) Expand all
30 30
31 private: 31 private:
32 // |mojo::ApplicationDelegate| override: 32 // |mojo::ApplicationDelegate| override:
33 bool ConfigureIncomingConnection( 33 bool ConfigureIncomingConnection(
34 mojo::ApplicationConnection* connection) override { 34 mojo::ApplicationConnection* connection) override {
35 connection->AddService<Process>(this); 35 connection->AddService<Process>(this);
36 return true; 36 return true;
37 } 37 }
38 38
39 // |InterfaceFactory<Process>| implementation: 39 // |InterfaceFactory<Process>| implementation:
40 void Create(mojo::ApplicationConnection* connection, 40 void Create(const mojo::ConnectionContext& connection_context,
41 mojo::InterfaceRequest<Process> request) override { 41 mojo::InterfaceRequest<Process> request) override {
42 if (!worker_pool_) { 42 if (!worker_pool_) {
43 worker_pool_ = new base::SequencedWorkerPool(kMaxWorkerThreads, 43 worker_pool_ = new base::SequencedWorkerPool(kMaxWorkerThreads,
44 "NativeSupportWorker"); 44 "NativeSupportWorker");
45 } 45 }
46 new ProcessImpl(worker_pool_, connection, request.Pass()); 46 new ProcessImpl(worker_pool_, connection_context, request.Pass());
47 } 47 }
48 48
49 scoped_refptr<base::SequencedWorkerPool> worker_pool_; 49 scoped_refptr<base::SequencedWorkerPool> worker_pool_;
50 50
51 DISALLOW_COPY_AND_ASSIGN(NativeSupportApp); 51 DISALLOW_COPY_AND_ASSIGN(NativeSupportApp);
52 }; 52 };
53 53
54 } // namespace native_support 54 } // namespace native_support
55 55
56 MojoResult MojoMain(MojoHandle application_request) { 56 MojoResult MojoMain(MojoHandle application_request) {
57 mojo::ApplicationRunnerChromium runner( 57 mojo::ApplicationRunnerChromium runner(
58 new native_support::NativeSupportApp()); 58 new native_support::NativeSupportApp());
59 // We need an I/O message loop, since we'll want to watch FDs. 59 // We need an I/O message loop, since we'll want to watch FDs.
60 runner.set_message_loop_type(base::MessageLoop::TYPE_IO); 60 runner.set_message_loop_type(base::MessageLoop::TYPE_IO);
61 return runner.Run(application_request); 61 return runner.Run(application_request);
62 } 62 }
OLDNEW
« no previous file with comments | « services/nacl/nonsfi/pnacl_link.cc ('k') | services/native_support/process_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698