[Video, Android] Pause video only players ignoring duration and keyframes

[Video, Android] Pause video only players ignoring duration and keyframes

My recent change https://codereview.chromium.org/2631633003 enabled
playing background video-only players on Android if they meet various
criteria like being shorter than an experimental limit or having
frequent enough keyframes.

Restore pausing logic by bypassing the irrelevant checks on Android.

BUG=None
TEST=Manual + updated unit tests

Review-Url: https://codereview.chromium.org/2643033004
Cr-Commit-Position: refs/heads/master@{#445116}
Committed: https://chromium.googlesource.com/chromium/src/+/ac1a85217d72b0043dfb41b447afe4f05d0d5588

[whywhat, 2017-01-20 00:49:11]

avayvod@chromium.org changed reviewers:
+ sandersd@chromium.org

[whywhat, 2017-01-20 00:49:17]

PTaL

[whywhat, 2017-01-20 00:49:30]

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

[commit-bot: I haz the power, 2017-01-20 00:51:12]

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-01-20 02:18:26]

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

[commit-bot: I haz the power, 2017-01-20 02:18:27]

Dry run: This issue passed the CQ dry run.

[whywhat, 2017-01-20 17:59:46]

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

[DaleCurtis, 2017-01-20 18:21:03]

lgtm

[whywhat, 2017-01-20 18:53:54]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2017-01-20 18:54:24]

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

[sandersd (OOO until July 31), 2017-01-20 19:01:19]

lgtm

[commit-bot: I haz the power, 2017-01-20 19:02:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484938434539940, "parent_rev":
"f174850ffb24a21a76cd786551203c26a47b97d4", "commit_rev":
"ac1a85217d72b0043dfb41b447afe4f05d0d5588"}

[commit-bot: I haz the power, 2017-01-20 19:04:02]

Description was changed from

==========
[Video, Android] Pause video only players ignoring duration and keyframes

My recent change https://codereview.chromium.org/2631633003 enabled
playing background video-only players on Android if they meet various
criteria like being shorter than an experimental limit or having
frequent enough keyframes.

Restore pausing logic by bypassing the irrelevant checks on Android.

BUG=None
TEST=Manual + updated unit tests
==========

to

==========
[Video, Android] Pause video only players ignoring duration and keyframes

My recent change https://codereview.chromium.org/2631633003 enabled
playing background video-only players on Android if they meet various
criteria like being shorter than an experimental limit or having
frequent enough keyframes.

Restore pausing logic by bypassing the irrelevant checks on Android.

BUG=None
TEST=Manual + updated unit tests

Review-Url: https://codereview.chromium.org/2643033004
Cr-Commit-Position: refs/heads/master@{#445116}
Committed:
https://chromium.googlesource.com/chromium/src/+/ac1a85217d72b0043dfb41b447af...
==========

[commit-bot: I haz the power, 2017-01-20 19:04:03]

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