Mojo: Don't allow closed pipe handles to be sent

Mojo: Don't allow closed pipe handles to be sent

Ports need to be sent, but a closed pipe's port no longer
exists. Sending a closed port = crashy times. This should be
prevented at the EDK layer where message serialization can
be sanely rejected. So do that.

This also adds a few CHECKS to help clarify other potential crashes.

BUG=587937
TBR=amistry@chromium.org

Committed: https://crrev.com/f44ffa817c0fffc2e7bb6c8bdcaf41e203a58d8a
Cr-Commit-Position: refs/heads/master@{#376392}

[Ken Rockot(use gerrit already), 2016-02-18 20:46:42]

Description was changed from

==========
Mojo: Don't allow closed pipe handles to be sent

Ports need to be sent, but a closed pipe's port no longer
exists. Sending a closed port = crashy times. This should be
prevented at the EDK layer where message serialization can
be sanely rejected. So do that.

This also adds a few CHECKS to help clarify other potential crashes.

BUG=587937
==========

to

==========
Mojo: Don't allow closed pipe handles to be sent

Ports need to be sent, but a closed pipe's port no longer
exists. Sending a closed port = crashy times. This should be
prevented at the EDK layer where message serialization can
be sanely rejected. So do that.

This also adds a few CHECKS to help clarify other potential crashes.

BUG=587937
TBR=amistry@chromium.org
==========

[Ken Rockot(use gerrit already), 2016-02-18 20:46:55]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-02-18 20:49:40]

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

[commit-bot: I haz the power, 2016-02-18 21:42:20]

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

[commit-bot: I haz the power, 2016-02-18 21:42:20]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[Ken Rockot(use gerrit already), 2016-02-19 04:21:41]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-02-19 04:23:38]

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

[commit-bot: I haz the power, 2016-02-19 05:08:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-19 05:10:06]

Description was changed from

==========
Mojo: Don't allow closed pipe handles to be sent

Ports need to be sent, but a closed pipe's port no longer
exists. Sending a closed port = crashy times. This should be
prevented at the EDK layer where message serialization can
be sanely rejected. So do that.

This also adds a few CHECKS to help clarify other potential crashes.

BUG=587937
TBR=amistry@chromium.org
==========

to

==========
Mojo: Don't allow closed pipe handles to be sent

Ports need to be sent, but a closed pipe's port no longer
exists. Sending a closed port = crashy times. This should be
prevented at the EDK layer where message serialization can
be sanely rejected. So do that.

This also adds a few CHECKS to help clarify other potential crashes.

BUG=587937
TBR=amistry@chromium.org

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

[commit-bot: I haz the power, 2016-02-19 05:10:07]

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