Explicitly serialise platform file attachments instead of doing them implicitly as a mojo handle at…

Explicitly serialise platform file attachments instead of doing them implicitly as a mojo handle attachment.

This change is intended to make it clearer how platform file attachments
are serialised and deserialised under ChannelMojo.

BUG=604282
Committed: https://crrev.com/980a61bb07fd46b7a2446cff29d302f252efa2b4
Cr-Commit-Position: refs/heads/master@{#398765}

[Anand Mistry (off Chromium), 2016-06-08 04:06:08]

amistry@chromium.org changed reviewers:
+ dcheng@chromium.org, rockot@chromium.org

[Anand Mistry (off Chromium), 2016-06-08 04:06:09]

dcheng: For .mojom change and in case there are any security concerns because
this changes how platform file attachments are treated under ChannelMojo.

[Anand Mistry (off Chromium), 2016-06-08 04:33:35]

Description was changed from

==========
Explicitly serialise platform file attachments instead of doing them implicitly
as a mojo handle attachment.

This change is intended to make it clearer how platform file attachments
are serialised and deserialised under ChannelMojo.

BUG=None
==========

to

==========
Explicitly serialise platform file attachments instead of doing them implicitly
as a mojo handle attachment.

This change is intended to make it clearer how platform file attachments
are serialised and deserialised under ChannelMojo.

BUG=604282
==========

[Ken Rockot(use gerrit already), 2016-06-08 17:45:29]

lgtm

[dcheng, 2016-06-08 18:16:28]

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

[dcheng, 2016-06-08 18:16:29]

+tsepez, who knows much more about IPC attachments than I do

[Tom Sepez, 2016-06-08 18:28:03]

lgtm

[Anand Mistry (off Chromium), 2016-06-08 23:29:39]

The CQ bit was checked by amistry@chromium.org

[commit-bot: I haz the power, 2016-06-08 23:29:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2047953002/1

[commit-bot: I haz the power, 2016-06-09 02:52:10]

Description was changed from

==========
Explicitly serialise platform file attachments instead of doing them implicitly
as a mojo handle attachment.

This change is intended to make it clearer how platform file attachments
are serialised and deserialised under ChannelMojo.

BUG=604282
==========

to

==========
Explicitly serialise platform file attachments instead of doing them implicitly
as a mojo handle attachment.

This change is intended to make it clearer how platform file attachments
are serialised and deserialised under ChannelMojo.

BUG=604282
==========

[commit-bot: I haz the power, 2016-06-09 02:52:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-09 02:55:11]

Description was changed from

==========
Explicitly serialise platform file attachments instead of doing them implicitly
as a mojo handle attachment.

This change is intended to make it clearer how platform file attachments
are serialised and deserialised under ChannelMojo.

BUG=604282
==========

to

==========
Explicitly serialise platform file attachments instead of doing them implicitly
as a mojo handle attachment.

This change is intended to make it clearer how platform file attachments
are serialised and deserialised under ChannelMojo.

BUG=604282

Committed: https://crrev.com/980a61bb07fd46b7a2446cff29d302f252efa2b4
Cr-Commit-Position: refs/heads/master@{#398765}
==========

[commit-bot: I haz the power, 2016-06-09 02:55:13]

Patchset 1 (id:??) landed as
https://crrev.com/980a61bb07fd46b7a2446cff29d302f252efa2b4
Cr-Commit-Position: refs/heads/master@{#398765}