Reduce boilerplate needed to configure mojo::Binding

mojo/public/cpp/bindings/binding.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 MOJO_PUBLIC_CPP_BINDINGS_BINDING_H_
 #define MOJO_PUBLIC_CPP_BINDINGS_BINDING_H_
 
 #include <utility>
 
 #include "base/macros.h"
@@ skipping 93 matching lines @@
           InterfaceRequest<GenericInterface> request,
           const MojoAsyncWaiter* waiter = Environment::GetDefaultAsyncWaiter())
       : Binding(impl) {
     Bind(request.PassMessagePipe(), waiter);
   }
 
   // Tears down the binding, closing the message pipe and leaving the interface
   // implementation unbound.
   ~Binding() {}
 
+  // Returns an InterfacePtr bound to one end of a pipe whose other end is
+  // bound to |this|.
+  InterfacePtr<Interface> CreateInterfacePtrAndBind() {

[yzshen1, 2016/01/21 01:12:09]

Please add a |waiter| param (just like line 127), and pass it into Bind().

[sky, 2016/01/21 14:35:26]

Do we really need the waiter param? And doesn't the style guide say no default
args? How about I add it if we actually use it somewhere.

+    InterfacePtr<Interface> interface_ptr;
+    Bind(&interface_ptr);
+    return std::move(interface_ptr);

[yzshen1, 2016/01/21 01:12:09]

I think std::move() is not needed.

[sky, 2016/01/21 17:50:24]

Done.

+  }
+
   // Completes a binding that was constructed with only an interface
   // implementation. Takes ownership of |handle| and binds it to the previously
   // specified implementation. See class comment for definition of |waiter|.
   void Bind(
       ScopedMessagePipeHandle handle,
       const MojoAsyncWaiter* waiter = Environment::GetDefaultAsyncWaiter()) {
     internal_state_.Bind(std::move(handle), waiter);
   }
 
   // Completes a binding that was constructed with only an interface
@@ skipping 115 matching lines @@
  private:
   internal::BindingState<Interface, Interface::PassesAssociatedKinds_>
       internal_state_;
 
   DISALLOW_COPY_AND_ASSIGN(Binding);
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_BINDING_H_