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

Side by Side Diff: mojo/application/public/cpp/lib/service_provider_impl.cc

Issue 1538823002: Convert Pass()→std::move() in mojo/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 "mojo/application/public/cpp/service_provider_impl.h" 5 #include <utility>
6 6
7 #include "mojo/application/public/cpp/service_connector.h" 7 #include "mojo/application/public/cpp/service_connector.h"
8 #include "mojo/application/public/cpp/service_provider_impl.h"
8 #include "mojo/public/cpp/environment/logging.h" 9 #include "mojo/public/cpp/environment/logging.h"
9 10
10 namespace mojo { 11 namespace mojo {
11 12
12 ServiceProviderImpl::ServiceProviderImpl() : binding_(this) { 13 ServiceProviderImpl::ServiceProviderImpl() : binding_(this) {
13 } 14 }
14 15
15 ServiceProviderImpl::ServiceProviderImpl( 16 ServiceProviderImpl::ServiceProviderImpl(
16 InterfaceRequest<ServiceProvider> request) 17 InterfaceRequest<ServiceProvider> request)
17 : binding_(this, request.Pass()) { 18 : binding_(this, std::move(request)) {}
18 }
19 19
20 ServiceProviderImpl::~ServiceProviderImpl() { 20 ServiceProviderImpl::~ServiceProviderImpl() {
21 } 21 }
22 22
23 void ServiceProviderImpl::Bind(InterfaceRequest<ServiceProvider> request) { 23 void ServiceProviderImpl::Bind(InterfaceRequest<ServiceProvider> request) {
24 binding_.Bind(request.Pass()); 24 binding_.Bind(std::move(request));
25 } 25 }
26 26
27 void ServiceProviderImpl::ConnectToService( 27 void ServiceProviderImpl::ConnectToService(
28 const String& service_name, 28 const String& service_name,
29 ScopedMessagePipeHandle client_handle) { 29 ScopedMessagePipeHandle client_handle) {
30 // TODO(beng): perhaps take app connection thru ctor so that we can pass 30 // TODO(beng): perhaps take app connection thru ctor so that we can pass
31 // ApplicationConnection through? 31 // ApplicationConnection through?
32 service_connector_registry_.ConnectToService(nullptr, service_name, 32 service_connector_registry_.ConnectToService(nullptr, service_name,
33 client_handle.Pass()); 33 std::move(client_handle));
34 } 34 }
35 35
36 void ServiceProviderImpl::SetServiceConnectorForName( 36 void ServiceProviderImpl::SetServiceConnectorForName(
37 ServiceConnector* service_connector, 37 ServiceConnector* service_connector,
38 const std::string& interface_name) { 38 const std::string& interface_name) {
39 service_connector_registry_.SetServiceConnectorForName(service_connector, 39 service_connector_registry_.SetServiceConnectorForName(service_connector,
40 interface_name); 40 interface_name);
41 } 41 }
42 42
43 } // namespace mojo 43 } // namespace mojo
OLDNEW
« no previous file with comments | « mojo/application/public/cpp/lib/service_connector_registry.cc ('k') | mojo/application/public/cpp/lib/service_registry.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698