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

Side by Side Diff: services/service_manager/runner/host/out_of_process_native_runner.cc

Issue 2566663004: Revert of Build services as standalone executables (Closed)
Patch Set: Created 4 years 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 #include "services/service_manager/runner/host/out_of_process_native_runner.h" 5 #include "services/service_manager/runner/host/out_of_process_native_runner.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <utility> 9 #include <utility>
10 10
11 #include "base/bind.h" 11 #include "base/bind.h"
12 #include "base/callback_helpers.h" 12 #include "base/callback_helpers.h"
13 #include "base/files/file_util.h" 13 #include "base/files/file_util.h"
14 #include "base/logging.h" 14 #include "base/logging.h"
15 #include "base/memory/ptr_util.h" 15 #include "base/memory/ptr_util.h"
16 #include "base/task_runner.h" 16 #include "base/task_runner.h"
17 #include "services/service_manager/runner/common/client_util.h" 17 #include "services/service_manager/runner/common/client_util.h"
18 #include "services/service_manager/runner/host/child_process_host.h" 18 #include "services/service_manager/runner/host/child_process_host.h"
19 #include "services/service_manager/runner/host/in_process_native_runner.h"
19 20
20 namespace service_manager { 21 namespace service_manager {
21 22
22 OutOfProcessNativeRunner::OutOfProcessNativeRunner( 23 OutOfProcessNativeRunner::OutOfProcessNativeRunner(
23 const base::FilePath& service_path,
24 base::TaskRunner* launch_process_runner, 24 base::TaskRunner* launch_process_runner,
25 NativeRunnerDelegate* delegate) 25 NativeRunnerDelegate* delegate)
26 : launch_process_runner_(launch_process_runner), 26 : launch_process_runner_(launch_process_runner), delegate_(delegate) {}
27 delegate_(delegate),
28 service_path_(service_path) {}
29 27
30 OutOfProcessNativeRunner::~OutOfProcessNativeRunner() { 28 OutOfProcessNativeRunner::~OutOfProcessNativeRunner() {
31 if (child_process_host_ && !service_path_.empty()) 29 if (child_process_host_ && !app_path_.empty())
32 child_process_host_->Join(); 30 child_process_host_->Join();
33 } 31 }
34 32
35 mojom::ServicePtr OutOfProcessNativeRunner::Start( 33 mojom::ServicePtr OutOfProcessNativeRunner::Start(
34 const base::FilePath& app_path,
36 const Identity& target, 35 const Identity& target,
37 bool start_sandboxed, 36 bool start_sandboxed,
38 const base::Callback<void(base::ProcessId)>& pid_available_callback, 37 const base::Callback<void(base::ProcessId)>& pid_available_callback,
39 const base::Closure& app_completed_callback) { 38 const base::Closure& app_completed_callback) {
39 app_path_ = app_path;
40
40 DCHECK(app_completed_callback_.is_null()); 41 DCHECK(app_completed_callback_.is_null());
41 app_completed_callback_ = app_completed_callback; 42 app_completed_callback_ = app_completed_callback;
42 43
43 child_process_host_.reset(new ChildProcessHost( 44 child_process_host_.reset(new ChildProcessHost(
44 launch_process_runner_, delegate_, start_sandboxed, target, 45 launch_process_runner_, delegate_, start_sandboxed, target, app_path));
45 service_path_));
46 return child_process_host_->Start( 46 return child_process_host_->Start(
47 target, pid_available_callback, 47 target, pid_available_callback,
48 base::Bind(&OutOfProcessNativeRunner::AppCompleted, 48 base::Bind(&OutOfProcessNativeRunner::AppCompleted,
49 base::Unretained(this))); 49 base::Unretained(this)));
50 } 50 }
51 51
52 void OutOfProcessNativeRunner::AppCompleted() { 52 void OutOfProcessNativeRunner::AppCompleted() {
53 if (child_process_host_) 53 if (child_process_host_)
54 child_process_host_->Join(); 54 child_process_host_->Join();
55 child_process_host_.reset(); 55 child_process_host_.reset();
56 // This object may be deleted by this callback. 56 // This object may be deleted by this callback.
57 base::Closure app_completed_callback = app_completed_callback_; 57 base::Closure app_completed_callback = app_completed_callback_;
58 app_completed_callback_.Reset(); 58 app_completed_callback_.Reset();
59 if (!app_completed_callback.is_null()) 59 if (!app_completed_callback.is_null())
60 app_completed_callback.Run(); 60 app_completed_callback.Run();
61 } 61 }
62 62
63 OutOfProcessNativeRunnerFactory::OutOfProcessNativeRunnerFactory( 63 OutOfProcessNativeRunnerFactory::OutOfProcessNativeRunnerFactory(
64 base::TaskRunner* launch_process_runner, 64 base::TaskRunner* launch_process_runner,
65 NativeRunnerDelegate* delegate) 65 NativeRunnerDelegate* delegate)
66 : launch_process_runner_(launch_process_runner), delegate_(delegate) {} 66 : launch_process_runner_(launch_process_runner), delegate_(delegate) {}
67
68 OutOfProcessNativeRunnerFactory::~OutOfProcessNativeRunnerFactory() {} 67 OutOfProcessNativeRunnerFactory::~OutOfProcessNativeRunnerFactory() {}
69 68
70 std::unique_ptr<NativeRunner> OutOfProcessNativeRunnerFactory::Create( 69 std::unique_ptr<NativeRunner> OutOfProcessNativeRunnerFactory::Create(
71 const base::FilePath& service_path) { 70 const base::FilePath& app_path) {
72 return base::MakeUnique<OutOfProcessNativeRunner>( 71 return base::MakeUnique<OutOfProcessNativeRunner>(launch_process_runner_,
73 service_path, launch_process_runner_, delegate_); 72 delegate_);
74 } 73 }
75 74
76 } // namespace service_manager 75 } // namespace service_manager
OLDNEW
« no previous file with comments | « services/service_manager/runner/host/out_of_process_native_runner.h ('k') | services/service_manager/service_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698