Remove dependency on //crypto in //ipc, except on windows.

//ipc was depending on //crypto just for crypto::RandBytes(), but
after crrev.com/347312 that function is used only on Windows, so
the //crypto is needed only on windows.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1321253011 .

Cr-Commit-Position: refs/heads/master@{#348638}

1 files changed, 18 insertions, 1 deletions

diff --git a/ipc/handle_attachment_win.cc b/ipc/handle_attachment_win.cc
index 50e3e6d..429c964 100644
--- a/ipc/handle_attachment_win.cc
+++ b/ipc/handle_attachment_win.cc
@@ -6,12 +6,29 @@
 
 #include <windows.h>
 
+#include "crypto/random.h"
+
 namespace IPC {
 namespace internal {
 
+namespace {
+
+// In order to prevent mutually untrusted processes from stealing resources from
+// one another, the nonce must be secret. This generates a 128-bit,
+// cryptographicaly-strong random number.
+BrokerableAttachment::AttachmentId GenerateAttachementId() {
+  BrokerableAttachment::AttachmentId result;
+  crypto::RandBytes(result.nonce, BrokerableAttachment::kNonceSize);
+  return result;
+}
+
+}  // namespace
+
 HandleAttachmentWin::HandleAttachmentWin(const HANDLE& handle,
                                          HandleWin::Permissions permissions)
-    : handle_(handle), permissions_(permissions) {}
+    : BrokerableAttachment(GenerateAttachementId(), true),
+      handle_(handle),
+      permissions_(permissions) {}
 
 HandleAttachmentWin::HandleAttachmentWin(const WireFormat& wire_format)
     : BrokerableAttachment(wire_format.attachment_id, false),