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

Unified Diff: ipc/ipc_message_attachment_set.cc

Issue 2473993003: Delete IPC::ChannelPosix, IPC::ChannelWin and IPC::AttachmentBroker. (Closed)
Patch Set: Created 4 years, 1 month 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 | « ipc/ipc_message_attachment_set.h ('k') | ipc/ipc_message_generator.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/ipc_message_attachment_set.cc
diff --git a/ipc/ipc_message_attachment_set.cc b/ipc/ipc_message_attachment_set.cc
index 3b7eefb9a6e0f1e5492fd09c8a0434f2af32510a..068e9d29c8f326bb17227881ef3ff494f01bd4de 100644
--- a/ipc/ipc_message_attachment_set.cc
+++ b/ipc/ipc_message_attachment_set.cc
@@ -184,23 +184,6 @@ MessageAttachmentSet::GetBrokerableAttachments() const {
return brokerable_attachments_;
}
-void MessageAttachmentSet::ReplacePlaceholderWithAttachment(
- const scoped_refptr<BrokerableAttachment>& attachment) {
- DCHECK_NE(BrokerableAttachment::PLACEHOLDER, attachment->GetBrokerableType());
- for (auto it = brokerable_attachments_.begin();
- it != brokerable_attachments_.end(); ++it) {
- if ((*it)->GetBrokerableType() == BrokerableAttachment::PLACEHOLDER &&
- (*it)->GetIdentifier() == attachment->GetIdentifier()) {
- *it = attachment;
- return;
- }
- }
-
- // This function should only be called if there is a placeholder ready to be
- // replaced.
- NOTREACHED();
-}
-
#if defined(OS_POSIX)
void MessageAttachmentSet::PeekDescriptors(base::PlatformFile* buffer) const {
« no previous file with comments | « ipc/ipc_message_attachment_set.h ('k') | ipc/ipc_message_generator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698