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

Unified Diff: mojo/edk/system/ports_message.h

Issue 2008953003: [mojo-edk] Explicitly serialise HANDLEs into messages instead of PlatformHandles. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove handles accessor Created 4 years, 7 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
« mojo/edk/system/node_channel.cc ('K') | « mojo/edk/system/node_controller.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/edk/system/ports_message.h
diff --git a/mojo/edk/system/ports_message.h b/mojo/edk/system/ports_message.h
index e2e2e8369264aa3efba36bd8a7cd7ed7f7e88ef5..bbc8e7954b6f38db4fdb357604413668e54ee784 100644
--- a/mojo/edk/system/ports_message.h
+++ b/mojo/edk/system/ports_message.h
@@ -25,7 +25,6 @@ class PortsMessage : public ports::Message {
~PortsMessage() override;
- PlatformHandle* handles() { return channel_message_->handles(); }
size_t num_handles() const { return channel_message_->num_handles(); }
bool has_handles() const { return channel_message_->has_handles(); }
@@ -33,6 +32,10 @@ class PortsMessage : public ports::Message {
channel_message_->SetHandles(std::move(handles));
}
+ ScopedPlatformHandleVectorPtr TakeHandles() {
+ return channel_message_->TakeHandles();
+ }
+
Channel::MessagePtr TakeChannelMessage() {
return std::move(channel_message_);
}
« mojo/edk/system/node_channel.cc ('K') | « mojo/edk/system/node_controller.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698