Use RunLoop, but do not call RunUntilIdle() multiple times

media/midi/midi_manager_unittest.cc

 // 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.
 
 #include "media/midi/midi_manager.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 57 matching lines @@
     wait_for_result_ = false;
   }
 
   virtual void ReceiveMidiData(uint32 port_index, const uint8* data,
                                size_t size, double timestamp) override {}
   virtual void AccumulateMidiBytesSent(size_t size) override {}
 
   int client_id() const { return client_id_; }
   MidiResult result() const { return result_; }
 
-  void HandleContinuationMessage() {
-    // Stop posting a dummy message once CompleteStartSession() is invoked.
-    if (!wait_for_result_)
-      return;
-    base::MessageLoop::current()->PostTask(
-        FROM_HERE,
-        base::Bind(&FakeMidiManagerClient::HandleContinuationMessage,
-                   base::Unretained(this)));
-  }
-
-
   MidiResult WaitForResult() {
-    base::RunLoop run_loop;
-    // Post a dummy task not to stop the following event loop.
-    HandleContinuationMessage();
-    // CompleteStartSession() is called inside the message loop on the same
-    // thread. Protection for |wait_for_result_| is not needed.
-    run_loop.RunUntilIdle();
+    while (wait_for_result_) {
+      base::RunLoop run_loop;
+      run_loop.RunUntilIdle();
+    }
     return result();
   }
 
  private:
   int client_id_;
   MidiResult result_;
   bool wait_for_result_;
 
   DISALLOW_COPY_AND_ASSIGN(FakeMidiManagerClient);
 };
@@ skipping 166 matching lines @@
   // Temporary until http://crbug.com/371230 is resolved.
   EXPECT_TRUE((result == MIDI_OK) || (result == MIDI_INITIALIZATION_ERROR));
 #else
   EXPECT_EQ(MIDI_OK, result);
 #endif
 }
 
 }  // namespace
 
 }  // namespace media