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

mojo/public/cpp/bindings/associated_interface_ptr.h

 // Copyright 2015 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_ASSOCIATED_INTERFACE_PTR_H_
 #define MOJO_PUBLIC_CPP_BINDINGS_ASSOCIATED_INTERFACE_PTR_H_
 
 #include <stdint.h>
 
 #include <string>
 #include <utility>
 
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
-#include "mojo/public/cpp/bindings/associated_group.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/connection_error_callback.h"
 #include "mojo/public/cpp/bindings/lib/associated_interface_ptr_state.h"
 #include "mojo/public/cpp/bindings/lib/multiplex_router.h"
 #include "mojo/public/cpp/system/message_pipe.h"
 
 namespace mojo {
 
 // Represents the client side of an associated interface. It is similar to
@@ skipping 123 matching lines @@
   // TODO: fix this restriction, it's not always obvious when there is a
   // pending response.
   AssociatedInterfacePtrInfo<Interface> PassInterface() {
     DCHECK(!internal_state_.has_pending_callbacks());
     State state;
     internal_state_.Swap(&state);
 
     return state.PassInterface();
   }
 
-  // Returns the associated group that this object belongs to. Returns null if
-  // the object is not bound.
-  AssociatedGroup* associated_group() {
-    return internal_state_.associated_group();
-  }
-
   // DO NOT USE. Exposed only for internal use and for testing.
   internal::AssociatedInterfacePtrState<Interface>* internal_state() {
     return &internal_state_;
   }
 
   // Allow AssociatedInterfacePtr<> to be used in boolean expressions, but not
   // implicitly convertible to a real bool (which is dangerous).
  private:
   // TODO(dcheng): Use an explicit conversion operator.
   typedef internal::AssociatedInterfacePtrState<Interface>
@@ skipping 19 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(AssociatedInterfacePtr);
 };
 
 // Creates an associated interface. The returned request is supposed to be sent
 // over another interface (either associated or non-associated).
 //
 // NOTE: |ptr| must NOT be used to make calls before the request is sent.
 // Violating that will lead to crash. On the other hand, as soon as the request
 // is sent, |ptr| is usable. There is no need to wait until the request is bound
 // to an implementation at the remote side.
-// TODO(yzshen): Remove the |group| parameter.
 template <typename Interface>
 AssociatedInterfaceRequest<Interface> MakeRequest(
     AssociatedInterfacePtr<Interface>* ptr,
-    AssociatedGroup* group = nullptr,
     scoped_refptr<base::SingleThreadTaskRunner> runner =
         base::ThreadTaskRunnerHandle::Get()) {
   AssociatedInterfacePtrInfo<Interface> ptr_info;
   auto request = MakeRequest(&ptr_info);
   ptr->Bind(std::move(ptr_info), std::move(runner));
   return request;
 }
 
 // Creates an associated interface. One of the two endpoints is supposed to be
 // sent over another interface (either associated or non-associated); while the
@@ skipping 62 matching lines @@
 // all callsites of this function to directly use that.
 template <typename Interface>
 AssociatedInterfaceRequest<Interface> MakeRequestForTesting(
     AssociatedInterfacePtr<Interface>* ptr) {
   return GetIsolatedProxy(ptr);
 }
 
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_ASSOCIATED_INTERFACE_PTR_H_