Unflaking PeerConnectionIntegrationTest.DtmfSenderObserver.

Unflaking PeerConnectionIntegrationTest.DtmfSenderObserver.

The test attempted to call InsertDtmf before verifying that the DTLS
handshake was complete. Unlike a data channel, the DTMF sender doesn't
do any buffering, so this isn't reliable.

BUG=webrtc:7547
NOTRY=True

Review-Url: https://codereview.webrtc.org/2855573004
Cr-Commit-Position: refs/heads/master@{#18041}
Committed: https://chromium.googlesource.com/external/webrtc/+/7145280954d4111230515d06e73f24c6d199b9ae

[Taylor Brandstetter, 2017-05-01 23:03:11]

deadbeef@webrtc.org changed reviewers:
+ pthatcher@webrtc.org

[Taylor Brandstetter, 2017-05-01 23:03:11]

PTAL

[Taylor Brandstetter, 2017-05-01 23:03:27]

Description was changed from

==========
Unflaking PeerConnectionIntegrationTest.DtmfSenderObserver.

The test attempted to call InsertDtmf before verifying that the DTLS
handshake was complete. Unlike a data channel, the DTMF sender doesn't
do any queueing, so this isn't reliable.

BUG=webrtc:7547
==========

to

==========
Unflaking PeerConnectionIntegrationTest.DtmfSenderObserver.

The test attempted to call InsertDtmf before verifying that the DTLS
handshake was complete. Unlike a data channel, the DTMF sender doesn't
do any buffering, so this isn't reliable.

BUG=webrtc:7547
==========

[pthatcher1, 2017-05-01 23:43:30]

lgtm

[Taylor Brandstetter, 2017-05-07 22:25:11]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2017-05-07 22:25:13]

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-05-07 23:30:41]

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

[commit-bot: I haz the power, 2017-05-07 23:30:42]

Try jobs failed on following builders:
  linux_msan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/20257)
  linux_tsan2 on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_tsan2/builds/22351)
  linux_ubsan_vptr on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_ubsan_vptr/builds...)

[Taylor Brandstetter, 2017-05-08 00:18:21]

Description was changed from

==========
Unflaking PeerConnectionIntegrationTest.DtmfSenderObserver.

The test attempted to call InsertDtmf before verifying that the DTLS
handshake was complete. Unlike a data channel, the DTMF sender doesn't
do any buffering, so this isn't reliable.

BUG=webrtc:7547
==========

to

==========
Unflaking PeerConnectionIntegrationTest.DtmfSenderObserver.

The test attempted to call InsertDtmf before verifying that the DTLS
handshake was complete. Unlike a data channel, the DTMF sender doesn't
do any buffering, so this isn't reliable.

BUG=webrtc:7547
NOTRY=True
==========

[Taylor Brandstetter, 2017-05-08 00:18:38]

On 2017/05/07 23:30:42, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   linux_msan on master.tryserver.webrtc (JOB_FAILED,
> http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/20257)
>   linux_tsan2 on master.tryserver.webrtc (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.webrtc/builders/linux_tsan2/builds/22351)
>   linux_ubsan_vptr on master.tryserver.webrtc (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.webrtc/builders/linux_ubsan_vptr/builds...)

Using NOTRY due to infra issues.

[Taylor Brandstetter, 2017-05-08 00:18:45]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2017-05-08 00:18:49]

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-05-08 00:21:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494202725154040, "parent_rev":
"6373774106648446496954ee5036a65492424fca", "commit_rev":
"7145280954d4111230515d06e73f24c6d199b9ae"}

[commit-bot: I haz the power, 2017-05-08 00:21:05]

Description was changed from

==========
Unflaking PeerConnectionIntegrationTest.DtmfSenderObserver.

The test attempted to call InsertDtmf before verifying that the DTLS
handshake was complete. Unlike a data channel, the DTMF sender doesn't
do any buffering, so this isn't reliable.

BUG=webrtc:7547
NOTRY=True
==========

to

==========
Unflaking PeerConnectionIntegrationTest.DtmfSenderObserver.

The test attempted to call InsertDtmf before verifying that the DTLS
handshake was complete. Unlike a data channel, the DTMF sender doesn't
do any buffering, so this isn't reliable.

BUG=webrtc:7547
NOTRY=True

Review-Url: https://codereview.webrtc.org/2855573004
Cr-Commit-Position: refs/heads/master@{#18041}
Committed:
https://chromium.googlesource.com/external/webrtc/+/7145280954d4111230515d06e...
==========

[commit-bot: I haz the power, 2017-05-08 00:21:06]

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