Allow video only elements to be paused even if they have large keyframe distance. |
|||
Issue descriptionTracking issue to merge https://codereview.chromium.org/2643033004 to M57. 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. The change is already submitted as: Commit: ac1a85217d72b0043dfb41b447afe4f05d0d5588 Cr-Commit-Position: refs/heads/master@{#445116}
,
Jan 24 2017
Merge approved for M57 branch 2987.
,
Jan 24 2017
Committed patchset #1 (id:1) manually as 47a2344d1df3addf7290e5d7ca4640f83e75d134. Description: [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} (cherry picked from commit ac1a85217d72b0043dfb41b447afe4f05d0d5588) Review-Url: https://codereview.chromium.org/2655723002 . Cr-Commit-Position: refs/branch-heads/2987@{#57} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} Committed: https://chromium.googlesource.com/chromium/src/+/47a2344d1df3addf7290e5d7ca4640f83e75d134 Affected files (+20, -2 lines): M media/blink/webmediaplayer_impl.cc M media/blink/webmediaplayer_impl_unittest.cc |
|||
►
Sign in to add a comment |
|||
Comment 1 by avayvod@chromium.org
, Jan 24 2017