Add option to set maximum video encoder bitrate to AppRTCDemo.

Add option to set maximum video encoder bitrate to AppRTCDemo.

BUG=b/30951236

Committed: https://crrev.com/a70856c0bf7e2817a68b82bdb131cfdd608d18f3
Cr-Commit-Position: refs/heads/master@{#14028}

[AlexG, 2016-08-31 22:50:07]

glaznev@webrtc.org changed reviewers:
+ wzh@webrtc.org

[AlexG, 2016-08-31 22:50:08]

PTAL

[AlexG, 2016-08-31 22:51:36]

The CQ bit was checked by glaznev@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-31 22:51:38]

Dry run: 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, 2016-09-01 00:52:12]

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

[commit-bot: I haz the power, 2016-09-01 00:52:13]

Dry run: Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[wzh, 2016-09-01 06:38:43]

lgtm

https://codereview.webrtc.org/2306433002/diff/1/webrtc/examples/androidapp/sr...
File webrtc/examples/androidapp/src/org/appspot/apprtc/PeerConnectionClient.java
(right):

https://codereview.webrtc.org/2306433002/diff/1/webrtc/examples/androidapp/sr...
webrtc/examples/androidapp/src/org/appspot/apprtc/PeerConnectionClient.java:885:
}
do you need an early return or a break here?

[AlexG, 2016-09-01 18:40:53]

On 2016/09/01 06:38:43, wzh wrote:
> lgtm
> 
>
https://codereview.webrtc.org/2306433002/diff/1/webrtc/examples/androidapp/sr...
> File
webrtc/examples/androidapp/src/org/appspot/apprtc/PeerConnectionClient.java
> (right):
> 
>
https://codereview.webrtc.org/2306433002/diff/1/webrtc/examples/androidapp/sr...
>
webrtc/examples/androidapp/src/org/appspot/apprtc/PeerConnectionClient.java:885:
> }
> do you need an early return or a break here?

Probably not - we are picking the latest sender

[AlexG, 2016-09-01 18:41:58]

The CQ bit was checked by glaznev@webrtc.org

[commit-bot: I haz the power, 2016-09-01 18:42:09]

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, 2016-09-01 18:49:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-01 18:49:57]

Description was changed from

==========
Add option to set maximum video encoder bitrate to AppRTCDemo.

BUG=b/30951236
==========

to

==========
Add option to set maximum video encoder bitrate to AppRTCDemo.

BUG=b/30951236

Committed: https://crrev.com/a70856c0bf7e2817a68b82bdb131cfdd608d18f3
Cr-Commit-Position: refs/heads/master@{#14028}
==========

[commit-bot: I haz the power, 2016-09-01 18:49:58]

Patchset 1 (id:??) landed as
https://crrev.com/a70856c0bf7e2817a68b82bdb131cfdd608d18f3
Cr-Commit-Position: refs/heads/master@{#14028}