Add option to disable particular HW video codec from app.

Add option to disable particular HW video codec from app.

Plus minor clean up / adding comments.

BUG=b/26695339
R=jiayl@webrtc.org

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

[AlexG, 2016-01-29 19:32:11]

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

[AlexG, 2016-01-29 19:32:11]

PTAL

[jiayl2, 2016-01-29 20:37:38]

https://codereview.webrtc.org/1644253003/diff/1/talk/app/webrtc/java/src/org/...
File talk/app/webrtc/java/src/org/webrtc/MediaCodecVideoEncoder.java (right):

https://codereview.webrtc.org/1644253003/diff/1/talk/app/webrtc/java/src/org/...
talk/app/webrtc/java/src/org/webrtc/MediaCodecVideoEncoder.java:146: public
static void disableVp8HwCodec() {
When should these methods be called?

[AlexG, 2016-01-29 20:39:49]

On 2016/01/29 20:37:38, jiayl2 wrote:
>
https://codereview.webrtc.org/1644253003/diff/1/talk/app/webrtc/java/src/org/...
> File talk/app/webrtc/java/src/org/webrtc/MediaCodecVideoEncoder.java (right):
> 
>
https://codereview.webrtc.org/1644253003/diff/1/talk/app/webrtc/java/src/org/...
> talk/app/webrtc/java/src/org/webrtc/MediaCodecVideoEncoder.java:146: public
> static void disableVp8HwCodec() {
> When should these methods be called?

Before PCF factory is created. At the same time when
WebRtcAudioManager.setBlacklistDeviceForOpenSLESUsage is called.

[jiayl2, 2016-01-29 21:11:54]

lgtm

[AlexG, 2016-01-29 21:12:34]

The CQ bit was checked by glaznev@webrtc.org

[commit-bot: I haz the power, 2016-01-29 21:12:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1644253003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1644253003/1

[AlexG, 2016-01-29 22:17:19]

Description was changed from

==========
Add option to disable particular HW video codec from app.

Plus minor clean up / adding comments.

BUG=b/26695339
==========

to

==========
Add option to disable particular HW video codec from app.

Plus minor clean up / adding comments.

BUG=b/26695339
R=jiayl@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/eee86a6aa3c172e079a451d05...
==========

[AlexG, 2016-01-29 22:17:20]

Committed patchset #1 (id:1) manually as
eee86a6aa3c172e079a451d05c0e17001e3bb4af (presubmit successful).

[commit-bot: I haz the power, 2016-01-29 22:17:22]

Description was changed from

==========
Add option to disable particular HW video codec from app.

Plus minor clean up / adding comments.

BUG=b/26695339
R=jiayl@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/eee86a6aa3c172e079a451d05...
==========

to

==========
Add option to disable particular HW video codec from app.

Plus minor clean up / adding comments.

BUG=b/26695339
R=jiayl@webrtc.org

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

[commit-bot: I haz the power, 2016-01-29 22:17:22]

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