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

Side by Side Diff: services/shell/public/cpp/lib/interface_provider.cc

Issue 2107143003: Revert of Make RenderProcessHost/RenderThread use MojoChildConnection/MojoShellConnection (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@rph2
Patch Set: Created 4 years, 5 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/shell/public/cpp/interface_provider.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/shell/public/cpp/interface_provider.h" 5 #include "services/shell/public/cpp/interface_provider.h"
6 6
7 namespace shell { 7 namespace shell {
8 8
9 InterfaceProvider::InterfaceProvider() : weak_factory_(this) { 9 InterfaceProvider::InterfaceProvider() : weak_factory_(this) {}
10 pending_request_ = GetProxy(&interface_provider_);
11 }
12 InterfaceProvider::~InterfaceProvider() {} 10 InterfaceProvider::~InterfaceProvider() {}
13 11
14 void InterfaceProvider::Bind(mojom::InterfaceProviderPtr interface_provider) { 12 void InterfaceProvider::Bind(mojom::InterfaceProviderPtr interface_provider) {
15 DCHECK(pending_request_.is_pending()); 13 DCHECK(!interface_provider_.is_bound());
16 mojo::FuseInterface(std::move(pending_request_), 14 interface_provider_ = std::move(interface_provider);
17 interface_provider.PassInterface());
18 } 15 }
19 16
20 void InterfaceProvider::SetConnectionLostClosure( 17 void InterfaceProvider::SetConnectionLostClosure(
21 const base::Closure& connection_lost_closure) { 18 const base::Closure& connection_lost_closure) {
22 interface_provider_.set_connection_error_handler(connection_lost_closure); 19 interface_provider_.set_connection_error_handler(connection_lost_closure);
23 } 20 }
24 21
25 base::WeakPtr<InterfaceProvider> InterfaceProvider::GetWeakPtr() { 22 base::WeakPtr<InterfaceProvider> InterfaceProvider::GetWeakPtr() {
26 return weak_factory_.GetWeakPtr(); 23 return weak_factory_.GetWeakPtr();
27 } 24 }
28 25
29 void InterfaceProvider::GetInterface( 26 void InterfaceProvider::GetInterface(
30 const std::string& name, 27 const std::string& name,
31 mojo::ScopedMessagePipeHandle request_handle) { 28 mojo::ScopedMessagePipeHandle request_handle) {
32 // Local binders can be registered via TestApi. 29 // Local binders can be registered via TestApi.
33 auto it = binders_.find(name); 30 auto it = binders_.find(name);
34 if (it != binders_.end()) { 31 if (it != binders_.end()) {
35 it->second.Run(std::move(request_handle)); 32 it->second.Run(std::move(request_handle));
36 return; 33 return;
37 } 34 }
38 interface_provider_->GetInterface(name, std::move(request_handle)); 35 interface_provider_->GetInterface(name, std::move(request_handle));
39 } 36 }
40 37
41 void InterfaceProvider::ClearBinders() { 38 void InterfaceProvider::ClearBinders() {
42 binders_.clear(); 39 binders_.clear();
43 } 40 }
44 41
45 } // namespace shell 42 } // namespace shell
OLDNEW
« no previous file with comments | « services/shell/public/cpp/interface_provider.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698