Adding Java wrapper for DtmfSender.

Adding Java wrapper for DtmfSender.

BUG=webrtc:4180
Review-Url: https://codereview.webrtc.org/2666873002
Cr-Commit-Position: refs/heads/master@{#16410}
Committed: https://chromium.googlesource.com/external/webrtc/+/24af66397e3e1919b801c8d56926263f90808ea1

[Taylor Brandstetter, 2017-01-31 08:32:34]

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

[magjed_webrtc, 2017-02-01 13:08:51]

lgtm % nits

https://codereview.webrtc.org/2666873002/diff/1/webrtc/sdk/android/api/org/we...
File webrtc/sdk/android/api/org/webrtc/DtmfSender.java (right):

https://codereview.webrtc.org/2666873002/diff/1/webrtc/sdk/android/api/org/we...
webrtc/sdk/android/api/org/webrtc/DtmfSender.java:21: // Returns true if this
DtmfSender is capable of sending DTMF.
nit: Use javadoc comments for the public methods. Also, 100 char limit.

https://codereview.webrtc.org/2666873002/diff/1/webrtc/sdk/android/api/org/we...
File webrtc/sdk/android/api/org/webrtc/RtpSender.java (right):

https://codereview.webrtc.org/2666873002/diff/1/webrtc/sdk/android/api/org/we...
webrtc/sdk/android/api/org/webrtc/RtpSender.java:20: private DtmfSender
dtmfSender;
Make this variable final.

[Taylor Brandstetter, 2017-02-01 17:31:28]

https://codereview.webrtc.org/2666873002/diff/1/webrtc/sdk/android/api/org/we...
File webrtc/sdk/android/api/org/webrtc/DtmfSender.java (right):

https://codereview.webrtc.org/2666873002/diff/1/webrtc/sdk/android/api/org/we...
webrtc/sdk/android/api/org/webrtc/DtmfSender.java:21: // Returns true if this
DtmfSender is capable of sending DTMF.
On 2017/02/01 13:08:51, magjed_webrtc wrote:
> nit: Use javadoc comments for the public methods. Also, 100 char limit.

Done.

https://codereview.webrtc.org/2666873002/diff/1/webrtc/sdk/android/api/org/we...
File webrtc/sdk/android/api/org/webrtc/RtpSender.java (right):

https://codereview.webrtc.org/2666873002/diff/1/webrtc/sdk/android/api/org/we...
webrtc/sdk/android/api/org/webrtc/RtpSender.java:20: private DtmfSender
dtmfSender;
On 2017/02/01 13:08:51, magjed_webrtc wrote:
> Make this variable final.

Done.

[pthatcher1, 2017-02-02 01:34:44]

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

[pthatcher1, 2017-02-02 01:34:45]

lgtm

[Taylor Brandstetter, 2017-02-02 05:10:31]

The CQ bit was checked by deadbeef@webrtc.org

[Taylor Brandstetter, 2017-02-02 05:10:31]

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

[commit-bot: I haz the power, 2017-02-02 05:10:33]

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 05:53:09]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486012231185180,
"parent_rev": "20cb0c1c85d4571919106019a562bbcbd35041fd", "commit_rev":
"24af66397e3e1919b801c8d56926263f90808ea1"}

[commit-bot: I haz the power, 2017-02-02 05:53:12]

Description was changed from

==========
Adding Java wrapper for DtmfSender.

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

to

==========
Adding Java wrapper for DtmfSender.

BUG=webrtc:4180

Review-Url: https://codereview.webrtc.org/2666873002
Cr-Commit-Position: refs/heads/master@{#16410}
Committed:
https://chromium.googlesource.com/external/webrtc/+/24af66397e3e1919b801c8d56...
==========

[commit-bot: I haz the power, 2017-02-02 05:53:13]

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