Fix a bunch of mojo_public_*_unittests with the new EDK.

mojo/public/cpp/bindings/tests/validation_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 <stddef.h>
 #include <stdint.h>
 #include <stdio.h>
 #include <algorithm>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/message_loop/message_loop.h"
+#include "base/run_loop.h"
 #include "mojo/message_pump/message_pump_mojo.h"
 #include "mojo/public/c/system/macros.h"
 #include "mojo/public/cpp/bindings/binding.h"
 #include "mojo/public/cpp/bindings/interface_ptr.h"
 #include "mojo/public/cpp/bindings/lib/connector.h"
 #include "mojo/public/cpp/bindings/lib/filter_chain.h"
 #include "mojo/public/cpp/bindings/lib/message_header_validator.h"
 #include "mojo/public/cpp/bindings/lib/router.h"
 #include "mojo/public/cpp/bindings/lib/validation_errors.h"
 #include "mojo/public/cpp/bindings/message.h"
@@ skipping 140 matching lines @@
   std::vector<std::string> names =
       EnumerateSourceRootRelativeDirectory(GetPath("", ""));
   std::vector<std::string> tests = GetMatchingTests(names, prefix);
 
   for (size_t i = 0; i < tests.size(); ++i) {
     Message message;
     std::string expected;
     ASSERT_TRUE(ReadTestCase(tests[i], &message, &expected));
 
     std::string result;
-    mojo::internal::ValidationErrorObserverForTesting observer;
+    base::RunLoop run_loop;
+    mojo::internal::ValidationErrorObserverForTesting observer(
+        run_loop.QuitClosure());
     mojo_ignore_result(test_message_receiver->Accept(&message));
+    if (expected != "PASS")  // Observer only gets called on errors.
+      run_loop.Run();
     if (observer.last_error() == mojo::internal::VALIDATION_ERROR_NONE)
       result = "PASS";
     else
       result = mojo::internal::ValidationErrorToString(observer.last_error());
 
     EXPECT_EQ(expected, result) << "failed test: " << tests[i];
   }
 }
 
 class DummyMessageReceiver : public MessageReceiver {
@@ skipping 42 matching lines @@
     TestMessageReceiver(ValidationIntegrationTest* owner,
                         ScopedMessagePipeHandle handle)
         : owner_(owner),
           connector_(std::move(handle),
                      mojo::internal::Connector::SINGLE_THREADED_SEND) {
       connector_.set_enforce_errors_from_incoming_receiver(false);
     }
     ~TestMessageReceiver() override {}
 
     bool Accept(Message* message) override {
-      bool rv = connector_.Accept(message);
-      owner_->PumpMessages();
-      return rv;
+      return connector_.Accept(message);
     }
 
    public:
     ValidationIntegrationTest* owner_;
     mojo::internal::Connector connector_;
   };
 
   void PumpMessages() { loop_.RunUntilIdle(); }
 
   base::MessageLoop loop_;
@@ skipping 237 matching lines @@
       static_cast<StructWithEnum::EnumWithin>(2)));
   EXPECT_TRUE(StructWithEnum::EnumWithin_IsValidValue(
       static_cast<StructWithEnum::EnumWithin>(3)));
   EXPECT_FALSE(StructWithEnum::EnumWithin_IsValidValue(
       static_cast<StructWithEnum::EnumWithin>(4)));
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace mojo