Reland "Mojo C++ bindings: support sync methods - part 2"

mojo/public/cpp/bindings/tests/router_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 "mojo/public/cpp/bindings/lib/router.h"
 
 #include <utility>
 
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
@@ skipping 21 matching lines @@
 
  protected:
   ScopedMessagePipeHandle handle0_;
   ScopedMessagePipeHandle handle1_;
 
  private:
   base::MessageLoop loop_;
 };
 
 TEST_F(RouterTest, BasicRequestResponse) {
-  internal::Router router0(std::move(handle0_), internal::FilterChain());
-  internal::Router router1(std::move(handle1_), internal::FilterChain());
+  internal::Router router0(std::move(handle0_), internal::FilterChain(), false);
+  internal::Router router1(std::move(handle1_), internal::FilterChain(), false);
 
   ResponseGenerator generator;
   router1.set_incoming_receiver(&generator);
 
   Message request;
   AllocRequestMessage(1, "hello", &request);
 
   MessageQueue message_queue;
   base::RunLoop run_loop;
   router0.AcceptWithResponder(
@@ skipping 22 matching lines @@
 
   EXPECT_FALSE(message_queue.IsEmpty());
 
   message_queue.Pop(&response);
 
   EXPECT_EQ(std::string("hello again world!"),
             std::string(reinterpret_cast<const char*>(response.payload())));
 }
 
 TEST_F(RouterTest, BasicRequestResponse_Synchronous) {
-  internal::Router router0(std::move(handle0_), internal::FilterChain());
-  internal::Router router1(std::move(handle1_), internal::FilterChain());
+  internal::Router router0(std::move(handle0_), internal::FilterChain(), false);
+  internal::Router router1(std::move(handle1_), internal::FilterChain(), false);
 
   ResponseGenerator generator;
   router1.set_incoming_receiver(&generator);
 
   Message request;
   AllocRequestMessage(1, "hello", &request);
 
   MessageQueue message_queue;
   router0.AcceptWithResponder(&request, new MessageAccumulator(&message_queue));
 
@@ skipping 20 matching lines @@
 
   EXPECT_FALSE(message_queue.IsEmpty());
 
   message_queue.Pop(&response);
 
   EXPECT_EQ(std::string("hello again world!"),
             std::string(reinterpret_cast<const char*>(response.payload())));
 }
 
 TEST_F(RouterTest, RequestWithNoReceiver) {
-  internal::Router router0(std::move(handle0_), internal::FilterChain());
-  internal::Router router1(std::move(handle1_), internal::FilterChain());
+  internal::Router router0(std::move(handle0_), internal::FilterChain(), false);
+  internal::Router router1(std::move(handle1_), internal::FilterChain(), false);
 
   // Without an incoming receiver set on router1, we expect router0 to observe
   // an error as a result of sending a message.
 
   Message request;
   AllocRequestMessage(1, "hello", &request);
 
   MessageQueue message_queue;
   base::RunLoop run_loop, run_loop2;
   router0.set_connection_error_handler(run_loop.QuitClosure());
   router1.set_connection_error_handler(run_loop2.QuitClosure());
   router0.AcceptWithResponder(&request, new MessageAccumulator(&message_queue));
 
   run_loop.Run();
   run_loop2.Run();
 
   EXPECT_TRUE(router0.encountered_error());
   EXPECT_TRUE(router1.encountered_error());
   EXPECT_TRUE(message_queue.IsEmpty());
 }
 
 // Tests Router using the LazyResponseGenerator. The responses will not be
 // sent until after the requests have been accepted.
 TEST_F(RouterTest, LazyResponses) {
-  internal::Router router0(std::move(handle0_), internal::FilterChain());
-  internal::Router router1(std::move(handle1_), internal::FilterChain());
+  internal::Router router0(std::move(handle0_), internal::FilterChain(), false);
+  internal::Router router1(std::move(handle1_), internal::FilterChain(), false);
 
   base::RunLoop run_loop;
   LazyResponseGenerator generator(run_loop.QuitClosure());
   router1.set_incoming_receiver(&generator);
 
   Message request;
   AllocRequestMessage(1, "hello", &request);
 
   MessageQueue message_queue;
   base::RunLoop run_loop2;
@@ skipping 44 matching lines @@
   message_queue.Pop(&response);
   EXPECT_EQ(std::string("hello again world!"),
             std::string(reinterpret_cast<const char*>(response.payload())));
 }
 
 // Tests that if the receiving application destroys the responder_ without
 // sending a response, then we trigger connection error at both sides. Moreover,
 // both sides still appear to have a valid message pipe handle bound.
 TEST_F(RouterTest, MissingResponses) {
   base::RunLoop run_loop0, run_loop1;
-  internal::Router router0(std::move(handle0_), internal::FilterChain());
+  internal::Router router0(std::move(handle0_), internal::FilterChain(), false);
   bool error_handler_called0 = false;
   router0.set_connection_error_handler(
       [&error_handler_called0, &run_loop0]() {
         error_handler_called0 = true;
         run_loop0.Quit();
       });
 
-  internal::Router router1(std::move(handle1_), internal::FilterChain());
+  internal::Router router1(std::move(handle1_), internal::FilterChain(), false);
   bool error_handler_called1 = false;
   router1.set_connection_error_handler(
       [&error_handler_called1, &run_loop1]() {
         error_handler_called1 = true;
         run_loop1.Quit();
       });
 
   base::RunLoop run_loop3;
   LazyResponseGenerator generator(run_loop3.QuitClosure());
   router1.set_incoming_receiver(&generator);
@@ skipping 31 matching lines @@
 }
 
 TEST_F(RouterTest, LateResponse) {
   // Test that things won't blow up if we try to send a message to a
   // MessageReceiver, which was given to us via AcceptWithResponder,
   // after the router has gone away.
 
   base::RunLoop run_loop;
   LazyResponseGenerator generator(run_loop.QuitClosure());
   {
-    internal::Router router0(std::move(handle0_), internal::FilterChain());
-    internal::Router router1(std::move(handle1_), internal::FilterChain());
+    internal::Router router0(std::move(handle0_), internal::FilterChain(),
+                             false);
+    internal::Router router1(std::move(handle1_), internal::FilterChain(),
+                             false);
 
     router1.set_incoming_receiver(&generator);
 
     Message request;
     AllocRequestMessage(1, "hello", &request);
 
     MessageQueue message_queue;
     router0.AcceptWithResponder(&request,
                                 new MessageAccumulator(&message_queue));
 
     run_loop.Run();
 
     EXPECT_TRUE(generator.has_responder());
   }
 
   EXPECT_FALSE(generator.responder_is_valid());
   generator.CompleteWithResponse();  // This should end up doing nothing.
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace mojo