Rationalize AddAwakable...() and RemoveAwakable...() methods.

mojo/edk/system/data_pipe_consumer_dispatcher.cc

 // Copyright 2013 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.
 
 #include "mojo/edk/system/data_pipe_consumer_dispatcher.h"
 
 #include <utility>
 
 #include "base/logging.h"
 #include "mojo/edk/system/data_pipe.h"
@@ skipping 176 matching lines @@
 }
 
 HandleSignalsState DataPipeConsumerDispatcher::GetHandleSignalsStateImplNoLock()
     const {
   mutex().AssertHeld();
   return data_pipe_->ConsumerGetHandleSignalsState();
 }
 
 MojoResult DataPipeConsumerDispatcher::AddAwakableImplNoLock(
     Awakable* awakable,
+    uint64_t context,
+    bool force,
     MojoHandleSignals signals,
-    bool force,
-    uint64_t context,
     HandleSignalsState* signals_state) {
   mutex().AssertHeld();
-  return data_pipe_->ConsumerAddAwakable(awakable, signals, force, context,
+  return data_pipe_->ConsumerAddAwakable(awakable, context, force, signals,
                                          signals_state);
 }
 
 void DataPipeConsumerDispatcher::RemoveAwakableImplNoLock(
-    Awakable* awakable,
-    HandleSignalsState* signals_state) {
-  mutex().AssertHeld();
-  data_pipe_->ConsumerRemoveAwakable(awakable, signals_state);
-}
-
-void DataPipeConsumerDispatcher::RemoveAwakableWithContextImplNoLock(
+    bool match_context,
     Awakable* awakable,
     uint64_t context,
     HandleSignalsState* signals_state) {
   mutex().AssertHeld();
-  data_pipe_->ConsumerRemoveAwakableWithContext(awakable, context,
-                                                signals_state);
+  data_pipe_->ConsumerRemoveAwakable(match_context, awakable, context,
+                                     signals_state);
 }
 
 void DataPipeConsumerDispatcher::StartSerializeImplNoLock(
     Channel* channel,
     size_t* max_size,
     size_t* max_platform_handles) {
   AssertHasOneRef();  // Only one ref => no need to take the lock.
   data_pipe_->ConsumerStartSerialize(channel, max_size, max_platform_handles);
 }
 
 bool DataPipeConsumerDispatcher::EndSerializeAndCloseImplNoLock(
     Channel* channel,
     void* destination,
     size_t* actual_size,
     std::vector<ScopedPlatformHandle>* platform_handles) {
   AssertHasOneRef();  // Only one ref => no need to take the lock.
 
   bool rv = data_pipe_->ConsumerEndSerialize(channel, destination, actual_size,
                                              platform_handles);
   data_pipe_ = nullptr;
   return rv;
 }
 
 }  // namespace system
 }  // namespace mojo