From 3722a32d89d1fa27b7510d82d14e94f0ea68aa9a Mon Sep 17 00:00:00 2001 From: erikchen Date: Wed, 7 Oct 2015 13:51:55 -0700 Subject: ipc: Update MachPortMac ownership semantics. This CL consists of a small refactor, a small change in ownership semantics, and a lot of documentation. The refactor removes a |const| qualifier from brokerable message attachments that are passed to the attachment broker. This allows for an improvement to ownership semantics for MachPortMac. BUG=535711 Review URL: https://codereview.chromium.org/1385143002 Cr-Commit-Position: refs/heads/master@{#352938} --- ipc/ipc_channel_reader.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'ipc/ipc_channel_reader.cc') diff --git a/ipc/ipc_channel_reader.cc b/ipc/ipc_channel_reader.cc index 5ca0e46..326c6d7 100644 --- a/ipc/ipc_channel_reader.cc +++ b/ipc/ipc_channel_reader.cc @@ -213,8 +213,8 @@ ChannelReader::AttachmentIdSet ChannelReader::GetBrokeredAttachments( #if USE_ATTACHMENT_BROKER MessageAttachmentSet* set = msg->attachment_set(); - std::vector brokerable_attachments_copy = - set->PeekBrokerableAttachments(); + std::vector brokerable_attachments_copy = + set->GetBrokerableAttachments(); for (const BrokerableAttachment* attachment : brokerable_attachments_copy) { if (attachment->NeedsBrokering()) { AttachmentBroker* broker = GetAttachmentBroker(); -- cgit v1.1