Mojo C++ bindings: remove unnecessary getters/params of AssociatedGroup.

mojo/public/cpp/bindings/associated_group.h

Index: mojo/public/cpp/bindings/associated_group.h
diff --git a/mojo/public/cpp/bindings/associated_group.h b/mojo/public/cpp/bindings/associated_group.h
index 21d6fcbf2dde4452ce1c90ce8898b42dd0dbca3a..14e78ec3f911c3280721bbab415d16bb6de2c469 100644
--- a/mojo/public/cpp/bindings/associated_group.h
+++ b/mojo/public/cpp/bindings/associated_group.h
@@ -7,8 +7,6 @@
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
-#include "mojo/public/cpp/bindings/associated_interface_ptr_info.h"
-#include "mojo/public/cpp/bindings/associated_interface_request.h"
 #include "mojo/public/cpp/bindings/bindings_export.h"
 #include "mojo/public/cpp/bindings/scoped_interface_endpoint_handle.h"
 
@@ -43,44 +41,6 @@ class MOJO_CPP_BINDINGS_EXPORT AssociatedGroup {
   //     remain unchanged ever since.
   AssociatedGroupController* GetController();
 
-  // TODO(yzshen): Remove the following public method. It is not needed anymore.
-  // Configuration used by CreateAssociatedInterface(). Please see the comments
-  // of that method for more details.
-  enum AssociatedInterfaceConfig { WILL_PASS_PTR, WILL_PASS_REQUEST };
-
-  // |config| indicates whether |ptr_info| or |request| will be sent to the
-  // remote side of the message pipe.
-  //
-  // NOTE: If |config| is |WILL_PASS_REQUEST|, you will want to bind |ptr_info|
-  // to a local AssociatedInterfacePtr to make calls. However, there is one
-  // restriction: the pointer should NOT be used to make calls before |request|
-  // is sent. Violating that will cause the message pipe to be closed. On the
-  // other hand, as soon as |request| is sent, the pointer is usable. There is
-  // no need to wait until |request| is bound to an implementation at the remote
-  // side.
-  template <typename T>
-  void CreateAssociatedInterface(AssociatedInterfaceConfig config,
-                                 AssociatedInterfacePtrInfo<T>* ptr_info,
-                                 AssociatedInterfaceRequest<T>* request) {
-    ScopedInterfaceEndpointHandle handle0;
-    ScopedInterfaceEndpointHandle handle1;
-    ScopedInterfaceEndpointHandle::CreatePairPendingAssociation(&handle0,
-                                                                &handle1);
-
-    ptr_info->set_handle(std::move(handle0));
-    request->Bind(std::move(handle1));
-
-    if (config == WILL_PASS_PTR) {
-      // The implementation is local, therefore set the version according to
-      // the interface definition that this code is built against.
-      ptr_info->set_version(T::Version_);
-    } else {
-      // The implementation is remote, we don't know about its actual version
-      // yet.
-      ptr_info->set_version(0u);
-    }
-  }
-
  private:
   base::Callback<AssociatedGroupController*()> controller_getter_;
   scoped_refptr<AssociatedGroupController> controller_;