Fix PlatformInfo variation for mime util IsCodecSupportedOnAndroidTest

Fix PlatformInfo variation for mime util IsCodecSupportedOnAndroidTest

The parameterized template function RunCodecSupportTest is supposed to
invoke tested callback with different combinations of PlatformInfo settings,
but it actually used only initial values of PlatformInfo fields.

Also it seems that H264 is always supported for non-encrypted video, so the
test IsCodecSupportedOnAndroidTest.ClearCodecBehavior should be corrected.
This has been revealed only after fixing parameter variation.

R=chcunningham, dalecurtis
BUG=709428

[chcunningham, 2017-04-08 00:03:36]

The CQ bit was checked by chcunningham@chromium.org to run a CQ dry run

[chcunningham, 2017-04-08 00:04:03]

LGTM. Thanks for fixing it!

[commit-bot: I haz the power, 2017-04-08 00:04:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-08 01:03:15]

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

[commit-bot: I haz the power, 2017-04-08 01:03:16]

Dry run: This issue passed the CQ dry run.

[pavor, 2017-04-10 09:12:20]

The CQ bit was checked by pavor@yandex-team.ru

[commit-bot: I haz the power, 2017-04-10 09:12:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-10 10:29:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491815540725080, "parent_rev":
"3317e61588169b796f87ded214021d2950ca0de6", "commit_rev":
"1b75c4d80c9ac27428c760c727c39500d6e8c27b"}

[commit-bot: I haz the power, 2017-04-10 10:30:25]

Description was changed from

==========
Fix PlatformInfo variation for mime util IsCodecSupportedOnAndroidTest

The parameterized template function RunCodecSupportTest is supposed to
invoke tested callback with different combinations of PlatformInfo settings,
but it actually used only initial values of PlatformInfo fields.

Also it seems that H264 is always supported for non-encrypted video, so the
test IsCodecSupportedOnAndroidTest.ClearCodecBehavior should be corrected.
This has been revealed only after fixing parameter variation.

R=chcunningham, dalecurtis
BUG=709428
==========

to

==========
Fix PlatformInfo variation for mime util IsCodecSupportedOnAndroidTest

The parameterized template function RunCodecSupportTest is supposed to
invoke tested callback with different combinations of PlatformInfo settings,
but it actually used only initial values of PlatformInfo fields.

Also it seems that H264 is always supported for non-encrypted video, so the
test IsCodecSupportedOnAndroidTest.ClearCodecBehavior should be corrected.
This has been revealed only after fixing parameter variation.

R=chcunningham, dalecurtis
BUG=709428

Review-Url: https://codereview.chromium.org/2804883007
Cr-Commit-Position: refs/heads/master@{#463213}
Committed:
https://chromium.googlesource.com/chromium/src/+/1b75c4d80c9ac27428c760c727c3...
==========

[commit-bot: I haz the power, 2017-04-10 10:30:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/1b75c4d80c9ac27428c760c727c3...

[Mike West, 2017-04-10 12:18:34]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2811693003/ by mkwst@chromium.org.

The reason for reverting is: Broke IsCodecSupportedOnAndroidTest on
https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20C....

[Mike West, 2017-04-10 12:19:32]

On 2017/04/10 at 12:18:34, Mike West wrote:
> A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2811693003/ by mkwst@chromium.org.
> 
> The reason for reverting is: Broke IsCodecSupportedOnAndroidTest on
https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20C....

Also
https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%....

[findit-for-me, 2017-04-10 12:27:19]

Findit identified this CL at revision 463213 as the culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

[pavor, 2017-04-11 11:37:38]

Description was changed from

==========
Fix PlatformInfo variation for mime util IsCodecSupportedOnAndroidTest

The parameterized template function RunCodecSupportTest is supposed to
invoke tested callback with different combinations of PlatformInfo settings,
but it actually used only initial values of PlatformInfo fields.

Also it seems that H264 is always supported for non-encrypted video, so the
test IsCodecSupportedOnAndroidTest.ClearCodecBehavior should be corrected.
This has been revealed only after fixing parameter variation.

R=chcunningham, dalecurtis
BUG=709428

Review-Url: https://codereview.chromium.org/2804883007
Cr-Commit-Position: refs/heads/master@{#463213}
Committed:
https://chromium.googlesource.com/chromium/src/+/1b75c4d80c9ac27428c760c727c3...
==========

to

==========
Fix PlatformInfo variation for mime util IsCodecSupportedOnAndroidTest

The parameterized template function RunCodecSupportTest is supposed to
invoke tested callback with different combinations of PlatformInfo settings,
but it actually used only initial values of PlatformInfo fields.

Also it seems that H264 is always supported for non-encrypted video, so the
test IsCodecSupportedOnAndroidTest.ClearCodecBehavior should be corrected.
This has been revealed only after fixing parameter variation.

R=chcunningham, dalecurtis
BUG=709428
==========