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

Unified Diff: mojo/edk/system/local_data_pipe_impl.cc

Issue 1396783004: Convert mojo::system::ChannelEndpointClient to use our new refcounting stuff (instead of base's). (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 2 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/edk/system/ipc_support_unittest.cc ('k') | mojo/edk/system/message_pipe.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/edk/system/local_data_pipe_impl.cc
diff --git a/mojo/edk/system/local_data_pipe_impl.cc b/mojo/edk/system/local_data_pipe_impl.cc
index 553d012e25a5facfaf36a521cfbd4ee74c47b971..1d1f205f26ef192ee0559a609ccc15a90378ef81 100644
--- a/mojo/edk/system/local_data_pipe_impl.cc
+++ b/mojo/edk/system/local_data_pipe_impl.cc
@@ -182,8 +182,9 @@ bool LocalDataPipeImpl::ProducerEndSerialize(
s->consumer_num_bytes = current_num_bytes_;
// Note: We don't use |port|.
RefPtr<ChannelEndpoint> channel_endpoint =
- channel->SerializeEndpointWithLocalPeer(destination_for_endpoint, nullptr,
- channel_endpoint_client(), 0);
+ channel->SerializeEndpointWithLocalPeer(
+ destination_for_endpoint, nullptr,
+ RefPtr<ChannelEndpointClient>(channel_endpoint_client()), 0);
// Note: Keep |*this| alive until the end of this method, to make things
// slightly easier on ourselves.
std::unique_ptr<DataPipeImpl> self(
@@ -360,9 +361,9 @@ bool LocalDataPipeImpl::ConsumerEndSerialize(
// Note: We don't use |port|.
RefPtr<ChannelEndpoint> channel_endpoint =
- channel->SerializeEndpointWithLocalPeer(destination_for_endpoint,
- &message_queue,
- channel_endpoint_client(), 0);
+ channel->SerializeEndpointWithLocalPeer(
+ destination_for_endpoint, &message_queue,
+ RefPtr<ChannelEndpointClient>(channel_endpoint_client()), 0);
// Note: Keep |*this| alive until the end of this method, to make things
// slightly easier on ourselves.
std::unique_ptr<DataPipeImpl> self(
« no previous file with comments | « mojo/edk/system/ipc_support_unittest.cc ('k') | mojo/edk/system/message_pipe.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698