Revert of Remove IPC::BrokerableAttachment.

Revert of Remove IPC::BrokerableAttachment. (patchset #1 id:120001 of https://codereview.chromium.org/2494943002/ )

Reason for revert:
Caused crash in NewlibPackagedAppTest.SuccessfulLoad browser_tests.

See http://crbug.com/665678#c1

BUG=665678
Original issue's description:
> Remove IPC::BrokerableAttachment.
>
> With only ChannelMojo in use, the distinction between brokerable and
> non-brokerable attachments no longer makes sense. This CL removes that
> distinction by removing BrokerableAttachment and flattening the
> hierarchy of attachment types.
>
> This also trims some POSIX-specific parts of IPC::MessageAttachmentSet.
>
> BUG=659448
>
> Committed: https://crrev.com/f6e03ce56c4d2370b79d0c3dd4ceb89cf5528e56
> Cr-Commit-Position: refs/heads/master@{#432153}

TBR=rockot@chromium.org,dcheng@chromium.org,mseaborn@chromium.org,raymes@chromium.org,erikchen@chromium.org,sammc@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=659448
Committed: https://crrev.com/cff1de9d81338681be3687697038553691b52c88
Cr-Commit-Position: refs/heads/master@{#432351}

[horo, 2016-11-16 01:44:33]

The CQ bit was checked by horo@chromium.org

[horo, 2016-11-16 01:44:33]

Created Revert of Remove IPC::BrokerableAttachment.

[commit-bot: I haz the power, 2016-11-16 01:44:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-16 01:53:24]

Description was changed from

==========
Revert of Remove IPC::BrokerableAttachment. (patchset #1 id:120001 of
https://codereview.chromium.org/2494943002/ )

Reason for revert:
Caused crash in NewlibPackagedAppTest.SuccessfulLoad browser_tests.

See http://crbug.com/665678#c1

BUG=665678

Original issue's description:
> Remove IPC::BrokerableAttachment.
> 
> With only ChannelMojo in use, the distinction between brokerable and
> non-brokerable attachments no longer makes sense. This CL removes that
> distinction by removing BrokerableAttachment and flattening the
> hierarchy of attachment types.
> 
> This also trims some POSIX-specific parts of IPC::MessageAttachmentSet.
> 
> BUG=659448
> 
> Committed: https://crrev.com/f6e03ce56c4d2370b79d0c3dd4ceb89cf5528e56
> Cr-Commit-Position: refs/heads/master@{#432153}

TBR=rockot@chromium.org,dcheng@chromium.org,mseaborn@chromium.org,raymes@chro...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=659448
==========

to

==========
Revert of Remove IPC::BrokerableAttachment. (patchset #1 id:120001 of
https://codereview.chromium.org/2494943002/ )

Reason for revert:
Caused crash in NewlibPackagedAppTest.SuccessfulLoad browser_tests.

See http://crbug.com/665678#c1

BUG=665678

Original issue's description:
> Remove IPC::BrokerableAttachment.
> 
> With only ChannelMojo in use, the distinction between brokerable and
> non-brokerable attachments no longer makes sense. This CL removes that
> distinction by removing BrokerableAttachment and flattening the
> hierarchy of attachment types.
> 
> This also trims some POSIX-specific parts of IPC::MessageAttachmentSet.
> 
> BUG=659448
> 
> Committed: https://crrev.com/f6e03ce56c4d2370b79d0c3dd4ceb89cf5528e56
> Cr-Commit-Position: refs/heads/master@{#432153}

TBR=rockot@chromium.org,dcheng@chromium.org,mseaborn@chromium.org,raymes@chro...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=659448
==========

[commit-bot: I haz the power, 2016-11-16 01:53:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-16 01:57:16]

Description was changed from

==========
Revert of Remove IPC::BrokerableAttachment. (patchset #1 id:120001 of
https://codereview.chromium.org/2494943002/ )

Reason for revert:
Caused crash in NewlibPackagedAppTest.SuccessfulLoad browser_tests.

See http://crbug.com/665678#c1

BUG=665678

Original issue's description:
> Remove IPC::BrokerableAttachment.
> 
> With only ChannelMojo in use, the distinction between brokerable and
> non-brokerable attachments no longer makes sense. This CL removes that
> distinction by removing BrokerableAttachment and flattening the
> hierarchy of attachment types.
> 
> This also trims some POSIX-specific parts of IPC::MessageAttachmentSet.
> 
> BUG=659448
> 
> Committed: https://crrev.com/f6e03ce56c4d2370b79d0c3dd4ceb89cf5528e56
> Cr-Commit-Position: refs/heads/master@{#432153}

TBR=rockot@chromium.org,dcheng@chromium.org,mseaborn@chromium.org,raymes@chro...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=659448
==========

to

==========
Revert of Remove IPC::BrokerableAttachment. (patchset #1 id:120001 of
https://codereview.chromium.org/2494943002/ )

Reason for revert:
Caused crash in NewlibPackagedAppTest.SuccessfulLoad browser_tests.

See http://crbug.com/665678#c1

BUG=665678

Original issue's description:
> Remove IPC::BrokerableAttachment.
>
> With only ChannelMojo in use, the distinction between brokerable and
> non-brokerable attachments no longer makes sense. This CL removes that
> distinction by removing BrokerableAttachment and flattening the
> hierarchy of attachment types.
>
> This also trims some POSIX-specific parts of IPC::MessageAttachmentSet.
>
> BUG=659448
>
> Committed: https://crrev.com/f6e03ce56c4d2370b79d0c3dd4ceb89cf5528e56
> Cr-Commit-Position: refs/heads/master@{#432153}

TBR=rockot@chromium.org,dcheng@chromium.org,mseaborn@chromium.org,raymes@chro...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=659448

Committed: https://crrev.com/cff1de9d81338681be3687697038553691b52c88
Cr-Commit-Position: refs/heads/master@{#432351}
==========

[commit-bot: I haz the power, 2016-11-16 01:57:17]

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