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

Unified Diff: third_party/mojo/src/mojo/edk/system/endpoint_relayer.cc

Issue 929433005: WIP: A couple of more ChannelMojo speedup ideas (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Build and test fix Created 5 years, 10 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
Index: third_party/mojo/src/mojo/edk/system/endpoint_relayer.cc
diff --git a/third_party/mojo/src/mojo/edk/system/endpoint_relayer.cc b/third_party/mojo/src/mojo/edk/system/endpoint_relayer.cc
index 52faefa4c6e783aaea5f95afb1290e7779f0b3e6..638e743ead6cec7640af42ac9a8c05a7208182f3 100644
--- a/third_party/mojo/src/mojo/edk/system/endpoint_relayer.cc
+++ b/third_party/mojo/src/mojo/edk/system/endpoint_relayer.cc
@@ -30,8 +30,10 @@ void EndpointRelayer::Init(ChannelEndpoint* endpoint0,
endpoints_[1] = endpoint1;
}
-bool EndpointRelayer::OnReadMessage(unsigned port, MessageInTransit* message) {
- DCHECK(message);
+bool EndpointRelayer::OnReadMessage(
+ unsigned port,
+ MessageInTransit::ReadContext& reading_message) {
+ DCHECK(!reading_message.passed());
base::AutoLock locker(lock_);
@@ -42,7 +44,7 @@ bool EndpointRelayer::OnReadMessage(unsigned port, MessageInTransit* message) {
// Otherwise, consume it even if the peer port is closed.
unsigned peer_port = GetPeerPort(port);
if (endpoints_[peer_port])
- endpoints_[peer_port]->EnqueueMessage(make_scoped_ptr(message));
+ endpoints_[peer_port]->EnqueueMessage(reading_message.PassMessage());
return true;
}
« no previous file with comments | « third_party/mojo/src/mojo/edk/system/endpoint_relayer.h ('k') | third_party/mojo/src/mojo/edk/system/incoming_endpoint.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698