Add a ServiceRegistry::ConnectToRemoteService that takes an InterfaceRequest<>.

content/public/common/service_registry.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_COMMON_SERVICE_REGISTRY_H_
 #define CONTENT_PUBLIC_COMMON_SERVICE_REGISTRY_H_
 
 #include <string>
 
 #include "base/bind.h"
@@ skipping 36 matching lines @@
   // connections to the service are unaffected.
   template <typename Interface>
   void RemoveService() {
     RemoveService(Interface::Name_);
   }
   virtual void RemoveService(const std::string& service_name) = 0;
 
   // Connect to an interface provided by the remote service provider.
   template <typename Interface>
   void ConnectToRemoteService(mojo::InterfacePtr<Interface>* ptr) {
-    ConnectToRemoteService(Interface::Name_,
-                           mojo::GetProxy(ptr).PassMessagePipe());
+    ConnectToRemoteService(mojo::GetProxy(ptr).Pass());

[Sam McNally, 2015/05/21 00:00:05]

Remove the .Pass(). mojo::GetProxy() already returns an rvalue.

[Anand Mistry (off Chromium), 2015/05/21 00:12:08]

Done.

+  }
+  template <typename Interface>
+  void ConnectToRemoteService(mojo::InterfaceRequest<Interface> ptr) {
+    ConnectToRemoteService(Interface::Name_, ptr.PassMessagePipe());
   }
   virtual void ConnectToRemoteService(const base::StringPiece& name,
                                       mojo::ScopedMessagePipeHandle handle) = 0;
 
  private:
   template <typename Interface>
   static void ForwardToServiceFactory(
       const base::Callback<void(mojo::InterfaceRequest<Interface>)>
           service_factory,
       mojo::ScopedMessagePipeHandle handle) {
     service_factory.Run(mojo::MakeRequest<Interface>(handle.Pass()));
   }
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_COMMON_SERVICE_REGISTRY_H_