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

Side by Side Diff: services/service_manager/public/cpp/lib/service_context.cc

Issue 2456493003: Add frame-specific InterfaceProviderSpec. (Closed)
Patch Set: . Created 4 years, 1 month 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/public/cpp/service_context.h" 5 #include "services/service_manager/public/cpp/service_context.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "mojo/public/cpp/bindings/interface_ptr.h" 10 #include "mojo/public/cpp/bindings/interface_ptr.h"
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 initialize_handler_.Run(); 56 initialize_handler_.Run();
57 57
58 callback.Run(std::move(pending_connector_request_)); 58 callback.Run(std::move(pending_connector_request_));
59 59
60 service_->OnStart(info); 60 service_->OnStart(info);
61 } 61 }
62 62
63 void ServiceContext::OnConnect( 63 void ServiceContext::OnConnect(
64 const ServiceInfo& source_info, 64 const ServiceInfo& source_info,
65 mojom::InterfaceProviderRequest interfaces) { 65 mojom::InterfaceProviderRequest interfaces) {
66 auto target_it = local_info_.interface_provider_specs.find( 66 InterfaceProviderSpec source_spec, target_spec;
67 mojom::kServiceManager_ConnectorSpec); 67 GetInterfaceProviderSpec(mojom::kServiceManager_ConnectorSpec,
68 InterfaceProviderSpec target_spec; 68 local_info_.interface_provider_specs, &target_spec);
69 if (target_it != local_info_.interface_provider_specs.end()) 69 GetInterfaceProviderSpec(mojom::kServiceManager_ConnectorSpec,
70 target_spec = target_it->second; 70 source_info.interface_provider_specs, &source_spec);
71 auto source_it = source_info.interface_provider_specs.find( 71 auto registry =
72 mojom::kServiceManager_ConnectorSpec); 72 base::MakeUnique<InterfaceRegistry>(mojom::kServiceManager_ConnectorSpec);
73 InterfaceProviderSpec source_spec; 73 registry->Bind(std::move(interfaces), local_info_.identity, target_spec,
74 if (source_it != source_info.interface_provider_specs.end()) 74 source_info.identity, source_spec);
75 source_spec = source_it->second;
76
77 auto registry = base::MakeUnique<InterfaceRegistry>(local_info_.identity,
78 target_spec);
79 registry->Bind(std::move(interfaces), source_info.identity, source_spec);
80 75
81 if (!service_->OnConnect(source_info, registry.get())) 76 if (!service_->OnConnect(source_info, registry.get()))
82 return; 77 return;
83 78
84 // TODO(beng): it appears we never prune this list. We should, when the 79 // TODO(beng): it appears we never prune this list. We should, when the
85 // registry's remote interface provider pipe breaks. 80 // registry's remote interface provider pipe breaks.
86 incoming_connections_.push_back(std::move(registry)); 81 incoming_connections_.push_back(std::move(registry));
87 } 82 }
88 83
89 //////////////////////////////////////////////////////////////////////////////// 84 ////////////////////////////////////////////////////////////////////////////////
90 // ServiceContext, private: 85 // ServiceContext, private:
91 86
92 void ServiceContext::OnConnectionError() { 87 void ServiceContext::OnConnectionError() {
93 // Note that the Service doesn't technically have to quit now, it may live 88 // Note that the Service doesn't technically have to quit now, it may live
94 // on to service existing connections. All existing Connectors however are 89 // on to service existing connections. All existing Connectors however are
95 // invalid. 90 // invalid.
96 should_run_connection_lost_closure_ = service_->OnStop(); 91 should_run_connection_lost_closure_ = service_->OnStop();
97 if (should_run_connection_lost_closure_ && 92 if (should_run_connection_lost_closure_ &&
98 !connection_lost_closure_.is_null()) 93 !connection_lost_closure_.is_null())
99 connection_lost_closure_.Run(); 94 connection_lost_closure_.Run();
100 // We don't reset the connector as clients may have taken a raw pointer to it. 95 // We don't reset the connector as clients may have taken a raw pointer to it.
101 // Connect() will return nullptr if they try to connect to anything. 96 // Connect() will return nullptr if they try to connect to anything.
102 } 97 }
103 98
104 } // namespace service_manager 99 } // namespace service_manager
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698