Move DTMF sender to RtpSender (as opposed to WebRtcSession).

Move DTMF sender to RtpSender (as opposed to WebRtcSession).

Previously in the spec, there was a createDtmfSender method on
PeerConnection, but that's been replaced by a "dtmf" attribute
on RtpSender, which allows getting a DTMF sender without having
an audio track.

This also simplifies the code slightly, since tracks are now not
necessary for identification.

BUG=webrtc:4180
Review-Url: https://codereview.webrtc.org/2666853002
Cr-Commit-Position: refs/heads/master@{#16409}
Committed: https://chromium.googlesource.com/external/webrtc/+/20cb0c1c85d4571919106019a562bbcbd35041fd

[Taylor Brandstetter, 2017-02-01 22:10:00]

https://codereview.webrtc.org/2666853002/diff/1/webrtc/pc/webrtcsession_unitt...
File webrtc/pc/webrtcsession_unittest.cc (left):

https://codereview.webrtc.org/2666853002/diff/1/webrtc/pc/webrtcsession_unitt...
webrtc/pc/webrtcsession_unittest.cc:3541: }
Moved to RtpSender tests.

[pthatcher1, 2017-02-02 01:33:37]

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

[pthatcher1, 2017-02-02 01:33:38]

lgtm

[Taylor Brandstetter, 2017-02-02 03:26:34]

The CQ bit was checked by deadbeef@webrtc.org

[Taylor Brandstetter, 2017-02-02 03:26:34]

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/2666853002/#ps40001 (title:
"Merge with master")

[commit-bot: I haz the power, 2017-02-02 03:26:44]

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-02-02 04:27:01]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486005994192820,
"parent_rev": "2e03c6611927af6ed08cf967b95342744ff329b6", "commit_rev":
"20cb0c1c85d4571919106019a562bbcbd35041fd"}

[commit-bot: I haz the power, 2017-02-02 04:27:03]

Description was changed from

==========
Move DTMF sender to RtpSender (as opposed to WebRtcSession).

Previously in the spec, there was a createDtmfSender method on
PeerConnection, but that's been replaced by a "dtmf" attribute
on RtpSender, which allows getting a DTMF sender without having
an audio track.

This also simplifies the code slightly, since tracks are now not
necessary for identification.

BUG=webrtc:4180
==========

to

==========
Move DTMF sender to RtpSender (as opposed to WebRtcSession).

Previously in the spec, there was a createDtmfSender method on
PeerConnection, but that's been replaced by a "dtmf" attribute
on RtpSender, which allows getting a DTMF sender without having
an audio track.

This also simplifies the code slightly, since tracks are now not
necessary for identification.

BUG=webrtc:4180

Review-Url: https://codereview.webrtc.org/2666853002
Cr-Commit-Position: refs/heads/master@{#16409}
Committed:
https://chromium.googlesource.com/external/webrtc/+/20cb0c1c85d4571919106019a...
==========

[commit-bot: I haz the power, 2017-02-02 04:27:04]

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