Fix flaky EndToEndTest.TransportSeqNumOnAudioAndVideo

Fix flaky EndToEndTest.TransportSeqNumOnAudioAndVideo
Add extra checks to it to simplify diagnostic should it fail again.

BUG=webrtc:7292
Review-Url: https://codereview.webrtc.org/2728103002
Cr-Commit-Position: refs/heads/master@{#16999}
Committed: https://chromium.googlesource.com/external/webrtc/+/bcb6004a9d53615a6428e070cbc6d52a2d8e833b

[danilchap, 2017-03-03 10:45:36]

danilchap@webrtc.org changed reviewers:
+ asapersson@webrtc.org

[danilchap, 2017-03-03 10:50:48]

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

[commit-bot: I haz the power, 2017-03-03 10:50:54]

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

[åsapersson, 2017-03-03 11:16:19]

lgtm

https://codereview.webrtc.org/2728103002/diff/1/webrtc/video/end_to_end_tests.cc
File webrtc/video/end_to_end_tests.cc (right):

https://codereview.webrtc.org/2728103002/diff/1/webrtc/video/end_to_end_tests...
webrtc/video/end_to_end_tests.cc:4066: received_packet_ids_.size() >= 50) {
maybe add a constant for 50

[commit-bot: I haz the power, 2017-03-03 11:40:25]

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

[commit-bot: I haz the power, 2017-03-03 11:40:26]

Dry run: This issue passed the CQ dry run.

[danilchap, 2017-03-03 12:14:03]

The CQ bit was checked by danilchap@webrtc.org

[danilchap, 2017-03-03 12:14:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
asapersson@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2728103002/#ps20001 (title:
"Name the kMinPacketsToWaitFor constant")

[commit-bot: I haz the power, 2017-03-03 12:14:11]

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

[danilchap, 2017-03-03 12:14:11]

https://codereview.webrtc.org/2728103002/diff/1/webrtc/video/end_to_end_tests.cc
File webrtc/video/end_to_end_tests.cc (right):

https://codereview.webrtc.org/2728103002/diff/1/webrtc/video/end_to_end_tests...
webrtc/video/end_to_end_tests.cc:4066: received_packet_ids_.size() >= 50) {
On 2017/03/03 11:16:19, åsapersson wrote:
> maybe add a constant for 50

Done.

[commit-bot: I haz the power, 2017-03-03 12:38:12]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488543243492470,
"parent_rev": "dea489f33e63c518854231fd3a98fabdb2b81563", "commit_rev":
"bcb6004a9d53615a6428e070cbc6d52a2d8e833b"}

[commit-bot: I haz the power, 2017-03-03 12:38:15]

Description was changed from

==========
Fix flaky EndToEndTest.TransportSeqNumOnAudioAndVideo
Add extra checks to it to simplify diagnostic should it fail again.

BUG=webrtc:7292
==========

to

==========
Fix flaky EndToEndTest.TransportSeqNumOnAudioAndVideo
Add extra checks to it to simplify diagnostic should it fail again.

BUG=webrtc:7292

Review-Url: https://codereview.webrtc.org/2728103002
Cr-Commit-Position: refs/heads/master@{#16999}
Committed:
https://chromium.googlesource.com/external/webrtc/+/bcb6004a9d53615a6428e070c...
==========

[commit-bot: I haz the power, 2017-03-03 12:38:15]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/bcb6004a9d53615a6428e070c...

[danilchap, 2017-03-03 13:04:36]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.webrtc.org/2730893002/ by danilchap@webrtc.org.

The reason for reverting is: Fails on tsan
http://build.chromium.org/p/client.webrtc/builders/Linux%20Tsan%20v2/builds/1....