EndToEndTest.ReceivesAndRetransmitsNack now handles reordering.

EndToEndTest.ReceivesAndRetransmitsNack now handles reordering.

Since packets can be received out of order NACKs could be triggered before any
packet was actually dropped. This cause the test to never drop packets which in
turn caused the block of code which set the |observation_complete_| event to
never execute.

BUG=webrtc:2845
Review-Url: https://codereview.webrtc.org/2613443002
Cr-Commit-Position: refs/heads/master@{#15990}
Committed: https://chromium.googlesource.com/external/webrtc/+/8054c7ecf890fc78c7a36032292d8a974c7dfb90

[philipel, 2017-01-03 13:51:23]

philipel@webrtc.org changed reviewers:
+ mflodman@webrtc.org

[philipel, 2017-01-03 14:59:54]

The CQ bit was checked by philipel@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-03 15:00:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-01-03 16:08:43]

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

[commit-bot: I haz the power, 2017-01-03 16:08:44]

Dry run: This issue passed the CQ dry run.

[philipel, 2017-01-09 13:53:11]

philipel@webrtc.org changed reviewers:
+ stefan@webrtc.org

[stefan-webrtc, 2017-01-10 12:14:00]

lgtm

[philipel, 2017-01-10 12:14:28]

The CQ bit was checked by philipel@webrtc.org

[commit-bot: I haz the power, 2017-01-10 12:14:31]

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

[commit-bot: I haz the power, 2017-01-10 13:19:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484050468935100, "parent_rev":
"11273f1e00f053cab875b6063410fe58a0b6b160", "commit_rev":
"8054c7ecf890fc78c7a36032292d8a974c7dfb90"}

[commit-bot: I haz the power, 2017-01-10 13:19:23]

Description was changed from

==========
EndToEndTest.ReceivesAndRetransmitsNack now handles reordering.

Since packets can be received out of order NACKs could be triggered before any
packet was actually dropped. This cause the test to never drop packets which in
turn caused the block of code which set the |observation_complete_| event to
never execute.

BUG=webrtc:2845
==========

to

==========
EndToEndTest.ReceivesAndRetransmitsNack now handles reordering.

Since packets can be received out of order NACKs could be triggered before any
packet was actually dropped. This cause the test to never drop packets which in
turn caused the block of code which set the |observation_complete_| event to
never execute.

BUG=webrtc:2845

Review-Url: https://codereview.webrtc.org/2613443002
Cr-Commit-Position: refs/heads/master@{#15990}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8054c7ecf890fc78c7a360322...
==========

[commit-bot: I haz the power, 2017-01-10 13:19:24]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/8054c7ecf890fc78c7a360322...