diff options
| author | erikchen <erikchen@chromium.org> | 2015-10-07 13:51:55 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-10-07 20:52:59 +0000 |
| commit | 3722a32d89d1fa27b7510d82d14e94f0ea68aa9a (patch) | |
| tree | 910333e192c721a279819064bd9349a788f08204 /ipc/ipc_channel_reader.cc | |
| parent | 4b0739defcac390878c6a692a387be2b3fb65e29 (diff) | |
| download | chromium_src-3722a32d89d1fa27b7510d82d14e94f0ea68aa9a.zip chromium_src-3722a32d89d1fa27b7510d82d14e94f0ea68aa9a.tar.gz chromium_src-3722a32d89d1fa27b7510d82d14e94f0ea68aa9a.tar.bz2 | |
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}
Diffstat (limited to 'ipc/ipc_channel_reader.cc')
| -rw-r--r-- | ipc/ipc_channel_reader.cc | 4 |
1 files changed, 2 insertions, 2 deletions
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<const BrokerableAttachment*> brokerable_attachments_copy = - set->PeekBrokerableAttachments(); + std::vector<BrokerableAttachment*> brokerable_attachments_copy = + set->GetBrokerableAttachments(); for (const BrokerableAttachment* attachment : brokerable_attachments_copy) { if (attachment->NeedsBrokering()) { AttachmentBroker* broker = GetAttachmentBroker(); |