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

Unified Diff: ipc/handle_win.cc

Issue 2494943002: Remove IPC::BrokerableAttachment. (Closed)
Patch Set: extra test output 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/handle_attachment_win.cc ('k') | ipc/ipc_channel_mojo.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/handle_win.cc
diff --git a/ipc/handle_win.cc b/ipc/handle_win.cc
index 60fa54c7cd1643cd08a3ed8093ab7d02d4f4e7b7..c8511e2017387ac523f238f6c8c76dae04f8c90d 100644
--- a/ipc/handle_win.cc
+++ b/ipc/handle_win.cc
@@ -38,16 +38,10 @@ bool ParamTraits<HandleWin>::Read(const base::Pickle* m,
return false;
MessageAttachment* attachment =
static_cast<MessageAttachment*>(base_attachment.get());
- if (attachment->GetType() != MessageAttachment::TYPE_BROKERABLE_ATTACHMENT)
+ if (attachment->GetType() != MessageAttachment::Type::WIN_HANDLE)
return false;
- BrokerableAttachment* brokerable_attachment =
- static_cast<BrokerableAttachment*>(attachment);
- if (brokerable_attachment->GetBrokerableType() !=
- BrokerableAttachment::WIN_HANDLE) {
- return false;
- }
IPC::internal::HandleAttachmentWin* handle_attachment =
- static_cast<IPC::internal::HandleAttachmentWin*>(brokerable_attachment);
+ static_cast<IPC::internal::HandleAttachmentWin*>(attachment);
r->set_handle(handle_attachment->get_handle());
handle_attachment->reset_handle_ownership();
return true;
« no previous file with comments | « ipc/handle_attachment_win.cc ('k') | ipc/ipc_channel_mojo.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698