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

Unified Diff: ipc/handle_attachment_win.h

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/brokerable_attachment.cc ('k') | ipc/handle_attachment_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/handle_attachment_win.h
diff --git a/ipc/handle_attachment_win.h b/ipc/handle_attachment_win.h
index cfe56a36bd81c95373b70d9f43f905f259d8f236..ef3b89652caf8c2e0fa332b3af659d6a3571545a 100644
--- a/ipc/handle_attachment_win.h
+++ b/ipc/handle_attachment_win.h
@@ -8,48 +8,16 @@
#include <stdint.h>
#include "base/process/process_handle.h"
-#include "ipc/brokerable_attachment.h"
#include "ipc/handle_win.h"
#include "ipc/ipc_export.h"
+#include "ipc/ipc_message_attachment.h"
namespace IPC {
namespace internal {
// This class represents a Windows HANDLE attached to a Chrome IPC message.
-class IPC_EXPORT HandleAttachmentWin : public BrokerableAttachment {
+class IPC_EXPORT HandleAttachmentWin : public MessageAttachment {
public:
- // The wire format for this handle.
- struct IPC_EXPORT WireFormat {
- // IPC translation requires that classes passed through IPC have a default
- // constructor.
- WireFormat()
- : handle(0),
- destination_process(0),
- permissions(HandleWin::INVALID) {}
-
- WireFormat(int32_t handle,
- const base::ProcessId& destination_process,
- HandleWin::Permissions permissions)
- : handle(handle),
- destination_process(destination_process),
- permissions(permissions) {}
-
- // The HANDLE that is intended for duplication, or the HANDLE that has been
- // duplicated, depending on context.
- // The type is int32_t instead of HANDLE because HANDLE gets typedefed to
- // void*, whose size varies between 32 and 64-bit processes. Using a
- // int32_t means that 64-bit processes will need to perform both up-casting
- // and down-casting. This is performed using the appropriate Windows APIs.
- // A value of 0 is equivalent to an invalid handle.
- int32_t handle;
-
- // The id of the destination process that the handle is duplicated into.
- base::ProcessId destination_process;
-
- // The permissions to use when duplicating the handle.
- HandleWin::Permissions permissions;
- };
-
// This constructor makes a copy of |handle| and takes ownership of the
// result. Should only be called by the sender of a Chrome IPC message.
HandleAttachmentWin(const HANDLE& handle, HandleWin::Permissions permissions);
@@ -61,14 +29,7 @@ class IPC_EXPORT HandleAttachmentWin : public BrokerableAttachment {
// receiver of a Chrome IPC message.
HandleAttachmentWin(const HANDLE& handle, FromWire from_wire);
- // This constructor takes ownership of |wire_format.handle| without making a
- // copy. Should only be called by the receiver of a Chrome IPC message.
- explicit HandleAttachmentWin(const WireFormat& wire_format);
-
- BrokerableType GetBrokerableType() const override;
-
- // Returns the wire format of this attachment.
- WireFormat GetWireFormat(const base::ProcessId& destination) const;
+ Type GetType() const override;
HANDLE get_handle() const { return handle_; }
« no previous file with comments | « ipc/brokerable_attachment.cc ('k') | ipc/handle_attachment_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698