Enable software decoding video with libvpx on Android

Enable software decoding video with libvpx on Android

VpxVideoDecoder previously would reject VP8 without an alpha
channel, leaving it to be decoded by FFmpegVideoDecoder. Now,
when it's compiled without FFmpegVideoDecoder it will handle
all VP8.

BUG=507834
Committed: https://crrev.com/dce82facb9614af36897f3c076e1842773871f1e
Cr-Commit-Position: refs/heads/master@{#342405}

[watk, 2015-08-05 22:19:52]

watk@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[DaleCurtis, 2015-08-05 22:39:24]

lgtm, did you want to include the media.gyp / build.gn change to enable
media_use_libvpx though?

[watk, 2015-08-05 23:01:39]

On 2015/08/05 22:39:24, DaleCurtis wrote:
> lgtm, did you want to include the media.gyp / build.gn change to enable
> media_use_libvpx though?

Hmm, I was just going to leave it as is for now (you have to set both
variables). I can change it so that media_use_ffmpeg=1 implies
media_use_libvpx=1 on Android?

[DaleCurtis, 2015-08-05 23:04:21]

Oh I see, yeah that seems fine with me. I forgot we deleted the build_ffmpegsumo
variable.

[DaleCurtis, 2015-08-05 23:29:52]

https://codereview.chromium.org/1281433002/diff/20001/media/media_variables.gypi
File media/media_variables.gypi (right):

https://codereview.chromium.org/1281433002/diff/20001/media/media_variables.g...
media/media_variables.gypi:27: ['(OS == "android" and media_use_ffmpeg == 0) or
OS == "ios"', {
I like your original idea of forcing media_use_libvpx for now and not this. Long
term we would just flip these defaults.

[watk, 2015-08-06 00:09:44]

Patchset #2 (id:20001) has been deleted

[watk, 2015-08-06 00:09:48]

The CQ bit was checked by watk@chromium.org

[commit-bot: I haz the power, 2015-08-06 00:10:19]

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

[commit-bot: I haz the power, 2015-08-06 01:59:27]

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

[commit-bot: I haz the power, 2015-08-06 01:59:27]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[watk, 2015-08-07 02:44:24]

The CQ bit was checked by watk@chromium.org

[commit-bot: I haz the power, 2015-08-07 02:45:37]

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

[commit-bot: I haz the power, 2015-08-07 03:57:48]

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

[commit-bot: I haz the power, 2015-08-07 03:57:49]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[watk, 2015-08-07 17:24:30]

The CQ bit was checked by watk@chromium.org

[commit-bot: I haz the power, 2015-08-07 17:24:48]

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

[commit-bot: I haz the power, 2015-08-07 19:17:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-08-07 19:18:09]

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