MediaCodecBridge: don't try to create .secure codecs pre-JB-MR2.

MediaCodecBridge: don't try to create .secure codecs pre-JB-MR2.

BUG=314868
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233272

[Ami GONE FROM CHROMIUM, 2013-11-06 02:08:02]

@wjia: please confirm that this fixes the crashes on RAZR & NexusS running 4.1.*

@xhwang: please confirm that this doesn't break EME for you :)

[wjia(left Chromium), 2013-11-06 02:23:27]

Thanks for the patch! It fixes the crash (apprtc.appspot.com/?debug=loopback).

LGTM with one comment.

https://codereview.chromium.org/60423004/diff/40001/media/base/android/java/s...
File media/base/android/java/src/org/chromium/media/MediaCodecBridge.java
(right):

https://codereview.chromium.org/60423004/diff/40001/media/base/android/java/s...
media/base/android/java/src/org/chromium/media/MediaCodecBridge.java:159: //
unavailable) so we side-step the issue.
Could you add the bug link here? Just for others' information once they wonder
why this workaround exists.

[Ami GONE FROM CHROMIUM, 2013-11-06 02:30:20]

On Tue, Nov 5, 2013 at 6:23 PM, <wjia@chromium.org> wrote:

> Could you add the bug link here? Just for others' information once they
> wonder why this workaround exists.
>

Done; thanks.

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[xhwang, 2013-11-06 02:31:08]

lgtm

[commit-bot: I haz the power, 2013-11-06 02:40:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/60423004/100001

[commit-bot: I haz the power, 2013-11-06 05:55:14]

Retried try job too often on win_rel for step(s) browser_tests, check_deps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-11-06 06:15:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/60423004/100001

[commit-bot: I haz the power, 2013-11-06 10:06:48]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-11-06 15:08:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/60423004/100001

[commit-bot: I haz the power, 2013-11-06 16:00:10]

Change committed as 233272