Avoid using MediaCodecList from Renderer process

Avoid using MediaCodecList from Renderer process

Renderer process cannot access the MediaServer process,
and it cannot access MediaCodecList.

Do not use getDefaultCodecName() in MediaCodecUtil.isDecoderSupportedForDevice,
instead,switch to Build.HARDWARE to check if the device is MTK.

We might hit some corner cases, but still better than page not loading.

BUG=604602, 597836
Committed: https://crrev.com/717ee19a82bde7395ea138843b09a826b5681463
Cr-Commit-Position: refs/heads/master@{#388304}

[qinmin, 2016-04-19 19:28:55]

qinmin@chromium.org changed reviewers:
+ timav@chromium.org

[qinmin, 2016-04-19 19:28:55]

PTAL

[Tima Vaisburd, 2016-04-19 19:35:20]

Description was changed from

==========
avoid using MediaCodecList from renderer

Renderer doesn't have any permissions, and it cannot access
MediaCodecList.
Switch to use Build.HARDWARE to check if the device is MTK.
We might hit some corner cases, but still better than page not loading.

BUG=604602
==========

to

==========
Avoid using MediaCodecList from Renderer process

Renderer process cannot access the MediaServer process,
and it cannot access MediaCodecList.

Do not use getDefaultCodecName() in MediaCodecUtil.isDecoderSupportedForDevice,
instead,
switch to Build.HARDWARE to check if the device is MTK.

We might hit some corner cases, but still better than page not loading.

BUG=604602, 597836
==========

[Tima Vaisburd, 2016-04-19 19:35:33]

Description was changed from

==========
Avoid using MediaCodecList from Renderer process

Renderer process cannot access the MediaServer process,
and it cannot access MediaCodecList.

Do not use getDefaultCodecName() in MediaCodecUtil.isDecoderSupportedForDevice,
instead,
switch to Build.HARDWARE to check if the device is MTK.

We might hit some corner cases, but still better than page not loading.

BUG=604602, 597836
==========

to

==========
Avoid using MediaCodecList from Renderer process

Renderer process cannot access the MediaServer process,
and it cannot access MediaCodecList.

Do not use getDefaultCodecName() in MediaCodecUtil.isDecoderSupportedForDevice,
instead,switch to Build.HARDWARE to check if the device is MTK.

We might hit some corner cases, but still better than page not loading.

BUG=604602, 597836
==========

[Tima Vaisburd, 2016-04-19 19:35:44]

lgtm

[qinmin, 2016-04-19 19:35:57]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2016-04-19 19:36:27]

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

[commit-bot: I haz the power, 2016-04-19 20:50:32]

Description was changed from

==========
Avoid using MediaCodecList from Renderer process

Renderer process cannot access the MediaServer process,
and it cannot access MediaCodecList.

Do not use getDefaultCodecName() in MediaCodecUtil.isDecoderSupportedForDevice,
instead,switch to Build.HARDWARE to check if the device is MTK.

We might hit some corner cases, but still better than page not loading.

BUG=604602, 597836
==========

to

==========
Avoid using MediaCodecList from Renderer process

Renderer process cannot access the MediaServer process,
and it cannot access MediaCodecList.

Do not use getDefaultCodecName() in MediaCodecUtil.isDecoderSupportedForDevice,
instead,switch to Build.HARDWARE to check if the device is MTK.

We might hit some corner cases, but still better than page not loading.

BUG=604602, 597836
==========

[commit-bot: I haz the power, 2016-04-19 20:50:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:16:10]

Description was changed from

==========
Avoid using MediaCodecList from Renderer process

Renderer process cannot access the MediaServer process,
and it cannot access MediaCodecList.

Do not use getDefaultCodecName() in MediaCodecUtil.isDecoderSupportedForDevice,
instead,switch to Build.HARDWARE to check if the device is MTK.

We might hit some corner cases, but still better than page not loading.

BUG=604602, 597836
==========

to

==========
Avoid using MediaCodecList from Renderer process

Renderer process cannot access the MediaServer process,
and it cannot access MediaCodecList.

Do not use getDefaultCodecName() in MediaCodecUtil.isDecoderSupportedForDevice,
instead,switch to Build.HARDWARE to check if the device is MTK.

We might hit some corner cases, but still better than page not loading.

BUG=604602, 597836

Committed: https://crrev.com/717ee19a82bde7395ea138843b09a826b5681463
Cr-Commit-Position: refs/heads/master@{#388304}
==========

[commit-bot: I haz the power, 2016-04-22 19:16:11]

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