ipc: Add methods for AttachmentBroker nonce serialization.

ipc: Add methods for AttachmentBroker nonce serialization.

BUG=493414
Committed: https://crrev.com/81352753aadf047281669c9c229bebe016310b4d
Cr-Commit-Position: refs/heads/master@{#347312}

[erikchen, 2015-08-28 20:10:15]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-08-28 20:10:52]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1321093003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1321093003/20001

[commit-bot: I haz the power, 2015-08-28 21:22:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-28 21:22:15]

Dry run: This issue passed the CQ dry run.

[erikchen, 2015-08-28 21:30:33]

erikchen@chromium.org changed reviewers:
+ tsepez@chromium.org

[erikchen, 2015-08-28 21:30:33]

tsepez: Please review.

This CL was broken out of "IPC: Add attachment brokering support to the message
header."
https://codereview.chromium.org/1303103002/

[Tom Sepez, 2015-09-01 17:54:23]

lgtm

https://codereview.chromium.org/1321093003/diff/20001/ipc/brokerable_attachme...
File ipc/brokerable_attachment.h (right):

https://codereview.chromium.org/1321093003/diff/20001/ipc/brokerable_attachme...
ipc/brokerable_attachment.h:25: // Default constructor returns a random nonce.
nit: ... unguessable random nonce.

[erikchen, 2015-09-04 01:13:31]

https://codereview.chromium.org/1321093003/diff/20001/ipc/brokerable_attachme...
File ipc/brokerable_attachment.h (right):

https://codereview.chromium.org/1321093003/diff/20001/ipc/brokerable_attachme...
ipc/brokerable_attachment.h:25: // Default constructor returns a random nonce.
On 2015/09/01 17:54:23, Tom Sepez wrote:
> nit: ... unguessable random nonce.

Done.

[erikchen, 2015-09-04 01:13:33]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2015-09-04 01:13:34]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/1321093003/#ps40001
(title: "Comments from tsepez.")

[commit-bot: I haz the power, 2015-09-04 01:13:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1321093003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1321093003/40001

[commit-bot: I haz the power, 2015-09-04 02:37:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-04 02:38:33]

Patchset 3 (id:??) landed as
https://crrev.com/81352753aadf047281669c9c229bebe016310b4d
Cr-Commit-Position: refs/heads/master@{#347312}