Improve the style of the objc wrapper for PeerConnectionInterface::SetBitrate.

Improve the style of the objc wrapper for PeerConnectionInterface::SetBitrate.

BUG=webrtc:7395
Review-Url: https://codereview.webrtc.org/3011013002
Cr-Commit-Position: refs/heads/master@{#19697}
Committed: https://chromium.googlesource.com/external/webrtc/+/8b476173d897c8e88330de683f9f9e9c81e76589

[Zach Stein, 2017-09-05 20:44:03]

zstein@webrtc.org changed reviewers:
+ tkchin@webrtc.org

[Zach Stein, 2017-09-05 20:44:03]

Addressing feedback from https://codereview.webrtc.org/2877933004/.

[tkchin_webrtc, 2017-09-05 20:51:51]

Thanks for addressing feedback.

lgtm % nits

https://codereview.webrtc.org/3011013002/diff/1/webrtc/sdk/objc/Framework/Hea...
File webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnection.h (right):

https://codereview.webrtc.org/3011013002/diff/1/webrtc/sdk/objc/Framework/Hea...
webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnection.h:188: *  Limits the
bandwidth allocated for all RTP streams sent by this
nit: move first word after /** to match methods above

https://codereview.webrtc.org/3011013002/diff/1/webrtc/sdk/objc/Framework/Hea...
webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnection.h:190: *
|currentBitrateBps| will force the available bitrate estimate to the given
for my own education - will bitrate estimate continue to change after setting
the current estimate?

[Zach Stein, 2017-09-05 21:14:41]

https://codereview.webrtc.org/3011013002/diff/1/webrtc/sdk/objc/Framework/Hea...
File webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnection.h (right):

https://codereview.webrtc.org/3011013002/diff/1/webrtc/sdk/objc/Framework/Hea...
webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnection.h:188: *  Limits the
bandwidth allocated for all RTP streams sent by this
On 2017/09/05 20:51:51, tkchin_webrtc wrote:
> nit: move first word after /** to match methods above

Done.

https://codereview.webrtc.org/3011013002/diff/1/webrtc/sdk/objc/Framework/Hea...
webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnection.h:190: *
|currentBitrateBps| will force the available bitrate estimate to the given
On 2017/09/05 20:51:51, tkchin_webrtc wrote:
> for my own education - will bitrate estimate continue to change after setting
> the current estimate?

Yes.

[Zach Stein, 2017-09-05 21:15:19]

The CQ bit was checked by zstein@webrtc.org

[Zach Stein, 2017-09-05 21:15:19]

The patchset sent to the CQ was uploaded after l-g-t-m from tkchin@webrtc.org
Link to the patchset: https://codereview.webrtc.org/3011013002/#ps20001 (title:
"Fix comment style.")

[commit-bot: I haz the power, 2017-09-05 21:15:29]

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-09-05 21:43:04]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1504646119482240,
"parent_rev": "e5abac443a5cbaf5acdcdc7c0876f8ba2aa48ad1", "commit_rev":
"8b476173d897c8e88330de683f9f9e9c81e76589"}

[commit-bot: I haz the power, 2017-09-05 21:43:09]

Description was changed from

==========
Improve the style of the objc wrapper for PeerConnectionInterface::SetBitrate.

BUG=webrtc:7395
==========

to

==========
Improve the style of the objc wrapper for PeerConnectionInterface::SetBitrate.

BUG=webrtc:7395

Review-Url: https://codereview.webrtc.org/3011013002
Cr-Commit-Position: refs/heads/master@{#19697}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8b476173d897c8e88330de683...
==========

[commit-bot: I haz the power, 2017-09-05 21:43:10]

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