ipc_channel: warn if someone forgets to check the result of Connect()

ipc/ipc_tests.cc

Index: ipc/ipc_tests.cc
diff --git a/ipc/ipc_tests.cc b/ipc/ipc_tests.cc
index a407c70a79e775af9e8108794c75d6ccdfc791b8..6edb393cb0f8be187170d85a626c85c1892b4a3d 100644
--- a/ipc/ipc_tests.cc
+++ b/ipc/ipc_tests.cc
@@ -219,7 +219,7 @@ TEST_F(IPCChannelTest, ChannelTest) {
   // Setup IPC channel.
   IPC::Channel chan(kTestClientChannel, IPC::Channel::MODE_SERVER,
                     &channel_listener);
-  chan.Connect();
+  ASSERT_TRUE(chan.Connect());
 
   channel_listener.Init(&chan);
 
@@ -334,7 +334,7 @@ TEST_F(IPCChannelTest, SendMessageInChannelConnected) {
   IPC::Channel channel(kTestClientChannel, IPC::Channel::MODE_SERVER,
                        &channel_listener);
   channel_listener.Init(&channel);
-  channel.Connect();
+  ASSERT_TRUE(channel.Connect());
 
   base::ProcessHandle process_handle = SpawnChild(TEST_CLIENT, &channel);
   ASSERT_TRUE(process_handle);
@@ -359,7 +359,7 @@ MULTIPROCESS_TEST_MAIN(RunTestClient) {
   // setup IPC channel
   IPC::Channel chan(kTestClientChannel, IPC::Channel::MODE_CLIENT,
                     &channel_listener);
-  chan.Connect();
+  ASSERT_TRUE(chan.Connect());
   channel_listener.Init(&chan);
   Send(&chan, "hello from child");
   // run message loop
@@ -492,7 +492,7 @@ TEST_F(IPCChannelTest, Performance) {
   IPC::Channel chan(kReflectorChannel, IPC::Channel::MODE_SERVER, NULL);
   ChannelPerfListener perf_listener(&chan, 10000, 100000);
   chan.set_listener(&perf_listener);
-  chan.Connect();
+  ASSERT_TRUE(chan.Connect());
 
   HANDLE process = SpawnChild(TEST_REFLECTOR, &chan);
   ASSERT_TRUE(process);
@@ -524,7 +524,7 @@ MULTIPROCESS_TEST_MAIN(RunReflector) {
   IPC::Channel chan(kReflectorChannel, IPC::Channel::MODE_CLIENT, NULL);
   ChannelReflectorListener channel_reflector_listener(&chan);
   chan.set_listener(&channel_reflector_listener);
-  chan.Connect();
+  ASSERT_TRUE(chan.Connect());
 
   MessageLoop::current()->Run();
   return true;