summaryrefslogtreecommitdiffstats
path: root/ipc/ipc_send_fds_test.cc
diff options
context:
space:
mode:
authormorrita@chromium.org <morrita@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-05-29 20:30:17 +0000
committermorrita@chromium.org <morrita@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-05-29 20:30:17 +0000
commit3909aa922a9e3c966de47ba2dde76c3a11596b43 (patch)
treee31523ae6de47f6e5ba253760e931829d619e8cc /ipc/ipc_send_fds_test.cc
parent7deb50808af310b10d9eb25ae81aefff8a3c652f (diff)
downloadchromium_src-3909aa922a9e3c966de47ba2dde76c3a11596b43.zip
chromium_src-3909aa922a9e3c966de47ba2dde76c3a11596b43.tar.gz
chromium_src-3909aa922a9e3c966de47ba2dde76c3a11596b43.tar.bz2
Introduce IPC::Channel::Create*() to ensure it being heap-allocated.
This change introduces IPC::Channel::Create*() API to turn IPC::Channel into a heap allocated object. This will allow us to make Channel a polymorphic class. This change also tries to hide Channel::Mode from public API so that we can simplify channel creation code paths cleaner in following changes. ChannelProxy has to follow same pattern to finish this cleanup. Such changes will follow. TEST=none BUG=377980 R=darin@chromium.org,cpu@chromium.org Review URL: https://codereview.chromium.org/307653003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273575 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ipc/ipc_send_fds_test.cc')
-rw-r--r--ipc/ipc_send_fds_test.cc16
1 files changed, 6 insertions, 10 deletions
diff --git a/ipc/ipc_send_fds_test.cc b/ipc/ipc_send_fds_test.cc
index aeec890..7e6a4e4 100644
--- a/ipc/ipc_send_fds_test.cc
+++ b/ipc/ipc_send_fds_test.cc
@@ -134,10 +134,10 @@ int SendFdsClientCommon(const std::string& test_client_name,
MyChannelDescriptorListener listener(expected_inode_num);
// Set up IPC channel.
- IPC::Channel channel(IPCTestBase::GetChannelName(test_client_name),
- IPC::Channel::MODE_CLIENT,
- &listener);
- CHECK(channel.Connect());
+ scoped_ptr<IPC::Channel> channel(IPC::Channel::CreateClient(
+ IPCTestBase::GetChannelName(test_client_name),
+ &listener));
+ CHECK(channel->Connect());
// Run message loop.
base::MessageLoop::current()->Run();
@@ -233,14 +233,10 @@ class PipeChannelHelper {
void Init() {
IPC::ChannelHandle in_handle("IN");
- in.reset(new IPC::Channel(in_handle,
- IPC::Channel::MODE_SERVER,
- &null_listener_));
+ in = IPC::Channel::CreateServer(in_handle, &null_listener_);
base::FileDescriptor out_fd(in->TakeClientFileDescriptor(), false);
IPC::ChannelHandle out_handle("OUT", out_fd);
- out.reset(new IPC::Channel(out_handle,
- IPC::Channel::MODE_CLIENT,
- &cb_listener_));
+ out = IPC::Channel::CreateClient(out_handle, &cb_listener_);
// PostTask the connect calls to make sure the callbacks happens
// on the right threads.
in_thread_->message_loop()->PostTask(