Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(664)

Unified Diff: mojo/system/remote_message_pipe_posix_unittest.cc

Issue 137273003: Mojo: Rename PlatformChannelHandle to PlatformHandle, etc. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mojo/system/raw_channel_win.cc ('k') | mojo/system/scoped_platform_handle.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/system/remote_message_pipe_posix_unittest.cc
diff --git a/mojo/system/remote_message_pipe_posix_unittest.cc b/mojo/system/remote_message_pipe_posix_unittest.cc
index 94196fe4f9abd5c059c0b3c15867eecbc98ae6a9..537392fd4dac43cf623e833aa3969fcf2dd7a58d 100644
--- a/mojo/system/remote_message_pipe_posix_unittest.cc
+++ b/mojo/system/remote_message_pipe_posix_unittest.cc
@@ -18,9 +18,9 @@
#include "mojo/system/channel.h"
#include "mojo/system/local_message_pipe_endpoint.h"
#include "mojo/system/message_pipe.h"
-#include "mojo/system/platform_channel.h"
#include "mojo/system/platform_channel_pair.h"
#include "mojo/system/proxy_message_pipe_endpoint.h"
+#include "mojo/system/scoped_platform_handle.h"
#include "mojo/system/test_utils.h"
#include "mojo/system/waiter.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -93,12 +93,8 @@ class RemoteMessagePipeTest : public testing::Test {
CHECK_EQ(base::MessageLoop::current(), io_thread_message_loop());
PlatformChannelPair channel_pair;
- platform_channels_[0] = channel_pair.CreateServerChannel();
- CHECK(platform_channels_[0].get());
- CHECK(platform_channels_[0]->is_valid());
- platform_channels_[1] = channel_pair.CreateClientChannel();
- CHECK(platform_channels_[1].get());
- CHECK(platform_channels_[1]->is_valid());
+ platform_handles_[0] = channel_pair.PassServerHandle();
+ platform_handles_[1] = channel_pair.PassClientHandle();
}
void CreateAndInitChannel(unsigned channel_index) {
@@ -108,7 +104,7 @@ class RemoteMessagePipeTest : public testing::Test {
channels_[channel_index] = new Channel();
CHECK(channels_[channel_index]->Init(
- platform_channels_[channel_index]->PassHandle()));
+ platform_handles_[channel_index].Pass()));
}
void ConnectMessagePipesOnIOThread(scoped_refptr<MessagePipe> mp_0,
@@ -156,7 +152,7 @@ class RemoteMessagePipeTest : public testing::Test {
}
base::Thread io_thread_;
- scoped_ptr<PlatformChannel> platform_channels_[2];
+ ScopedPlatformHandle platform_handles_[2];
scoped_refptr<Channel> channels_[2];
DISALLOW_COPY_AND_ASSIGN(RemoteMessagePipeTest);
« no previous file with comments | « mojo/system/raw_channel_win.cc ('k') | mojo/system/scoped_platform_handle.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698