[mojo-edk] Revert ObserveProxy retransmission behavior

[mojo-edk] Revert ObserveProxy retransmission behavior

Gets rid of ObserveProxy retransmission which was only required
to facilitate excessive randomization in tests. In practice this
behavior can race with normal port closure and lead to busy
loops.

Gets rid of excessive randomization in tests. The new PortsTest
environment more closely emulates reality: Nodes operate
independently but there are some basic and trivially satisfiable
ordering guarantees.

Gets rid of deferred AcceptIncomingMessages pumping in
NodeController, which was only added to facilitate
ObserveProxy retransmission.

BUG=628492, 635508
Review-Url: https://codereview.chromium.org/2219733005
Cr-Commit-Position: refs/heads/master@{#410535}
(cherry picked from commit 36f85398960e7836d5325e1a58298209ced34f33)

Committed: https://chromium.googlesource.com/chromium/src/+/573af98298255787657627c25a78a5de1676da73

[Ken Rockot(use gerrit already), 2016-08-10 23:19:36]

Description was changed from

==========
[mojo-edk] Revert ObserveProxy retransmission behavior

Gets rid of ObserveProxy retransmission which was only required
to facilitate excessive randomization in tests. In practice this
behavior can race with normal port closure and lead to busy
loops.

Gets rid of excessive randomization in tests. The new PortsTest
environment more closely emulates reality: Nodes operate
independently but there are some basic and trivially satisfiable
ordering guarantees.

Gets rid of deferred AcceptIncomingMessages pumping in
NodeController, which was only added to facilitate
ObserveProxy retransmission.

BUG=628492,635508

Review-Url: https://codereview.chromium.org/2219733005
Cr-Commit-Position: refs/heads/master@{#410535}
(cherry picked from commit 36f85398960e7836d5325e1a58298209ced34f33)
==========

to

==========
[mojo-edk] Revert ObserveProxy retransmission behavior

Gets rid of ObserveProxy retransmission which was only required
to facilitate excessive randomization in tests. In practice this
behavior can race with normal port closure and lead to busy
loops.

Gets rid of excessive randomization in tests. The new PortsTest
environment more closely emulates reality: Nodes operate
independently but there are some basic and trivially satisfiable
ordering guarantees.

Gets rid of deferred AcceptIncomingMessages pumping in
NodeController, which was only added to facilitate
ObserveProxy retransmission.

BUG=628492,635508

Review-Url: https://codereview.chromium.org/2219733005
Cr-Commit-Position: refs/heads/master@{#410535}
(cherry picked from commit 36f85398960e7836d5325e1a58298209ced34f33)

Committed:
https://chromium.googlesource.com/chromium/src/+/573af98298255787657627c25a78...
==========

[Ken Rockot(use gerrit already), 2016-08-10 23:19:37]

Committed patchset #1 (id:1) manually as
573af98298255787657627c25a78a5de1676da73.