Only set certificate on DTLS transport if fingerprint is found in SDP.

Only set certificate on DTLS transport if fingerprint is found in SDP.

This is used for fallback from DTLS to SDES encryption, which we probably still
want to support. Setting a certificate puts the DTLS transport in a "DTLS
enabled" mode, so it should be delayed until SDP with "a=fingerprint" is set.

BUG=webrtc:6972
Review-Url: https://codereview.webrtc.org/2641633002
Cr-Commit-Position: refs/heads/master@{#16199}
Committed: https://chromium.googlesource.com/external/webrtc/+/8662f940230bb0f8989ddc993488cde3a9c5b76a

[pthatcher1, 2017-01-20 19:31:55]

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

[pthatcher1, 2017-01-20 19:31:56]

lgtm

[Taylor Brandstetter, 2017-01-21 02:38:53]

The CQ bit was checked by deadbeef@webrtc.org

[Taylor Brandstetter, 2017-01-21 02:38:53]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2641633002/#ps20001 (title:
"Merge with master")

[commit-bot: I haz the power, 2017-01-21 02:38:59]

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-21 02:46:49]

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

[commit-bot: I haz the power, 2017-01-21 02:46:50]

Try jobs failed on following builders:
  linux_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_rel/builds/22253)
  linux_ubsan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_ubsan/builds/8830)

[Taylor Brandstetter, 2017-01-21 04:12:51]

The CQ bit was checked by deadbeef@webrtc.org

[Taylor Brandstetter, 2017-01-21 04:12:51]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2641633002/#ps40001 (title:
"Fix tests by making FakeTransportController::Connect put fingerprints in
transport descriptions.")

[commit-bot: I haz the power, 2017-01-21 04:12:52]

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-21 05:20:52]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484971971035600,
"parent_rev": "2197e9186087012fb677a5865a42835297cdea19", "commit_rev":
"8662f940230bb0f8989ddc993488cde3a9c5b76a"}

[commit-bot: I haz the power, 2017-01-21 05:20:54]

Description was changed from

==========
Only set certificate on DTLS transport if fingerprint is found in SDP.

This is used for fallback from DTLS to SDES encryption, which we probably still
want to support. Setting a certificate puts the DTLS transport in a "DTLS
enabled" mode, so it should be delayed until SDP with "a=fingerprint" is set.

BUG=webrtc:6972
==========

to

==========
Only set certificate on DTLS transport if fingerprint is found in SDP.

This is used for fallback from DTLS to SDES encryption, which we probably still
want to support. Setting a certificate puts the DTLS transport in a "DTLS
enabled" mode, so it should be delayed until SDP with "a=fingerprint" is set.

BUG=webrtc:6972

Review-Url: https://codereview.webrtc.org/2641633002
Cr-Commit-Position: refs/heads/master@{#16199}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8662f940230bb0f8989ddc993...
==========

[commit-bot: I haz the power, 2017-01-21 05:20:55]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/8662f940230bb0f8989ddc993...