diff options
| author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-19 19:11:15 +0000 |
|---|---|---|
| committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-19 19:11:15 +0000 |
| commit | 39703fb7cbda4ac831258c70e7fac6517125febc (patch) | |
| tree | 8460f0622922154910721c7d2a698a0ea52106e5 /ipc/ipc_send_fds_test.cc | |
| parent | 63dc378eac88be2da11255b61d7af140bb89ba55 (diff) | |
| download | chromium_src-39703fb7cbda4ac831258c70e7fac6517125febc.zip chromium_src-39703fb7cbda4ac831258c70e7fac6517125febc.tar.gz chromium_src-39703fb7cbda4ac831258c70e7fac6517125febc.tar.bz2 | |
ipc_channel: warn if someone forgets to check the result of Connect()
I found a test that was doing this.
Review URL: http://codereview.chromium.org/3759004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63083 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ipc/ipc_send_fds_test.cc')
| -rw-r--r-- | ipc/ipc_send_fds_test.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ipc/ipc_send_fds_test.cc b/ipc/ipc_send_fds_test.cc index e1f19dc..183a06c 100644 --- a/ipc/ipc_send_fds_test.cc +++ b/ipc/ipc_send_fds_test.cc @@ -110,7 +110,7 @@ int TestDescriptorClient(ino_t expected_inode_num) { // Setup IPC channel. IPC::Channel chan(kTestClientChannel, IPC::Channel::MODE_CLIENT, &listener); - chan.Connect(); + CHECK(chan.Connect()); MessageLoop::current()->Run(); return 0; @@ -157,7 +157,7 @@ TEST_F(IPCChannelTest, DescriptorTestSandboxed) { IPC::Channel chan(kTestClientChannel, IPC::Channel::MODE_SERVER, &listener); - chan.Connect(); + ASSERT_TRUE(chan.Connect()); base::ProcessHandle process_handle = SpawnChild( TEST_DESCRIPTOR_CLIENT_SANDBOXED, @@ -181,7 +181,7 @@ TEST_F(IPCChannelTest, DescriptorTest) { IPC::Channel chan(kTestClientChannel, IPC::Channel::MODE_SERVER, &listener); - chan.Connect(); + ASSERT_TRUE(chan.Connect()); base::ProcessHandle process_handle = SpawnChild(TEST_DESCRIPTOR_CLIENT, &chan); |