New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 607918 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug

Blocking:
issue 487345



Sign in to add a comment

Autoplay experiment related to visibility should only apply to vidieo

Project Member Reported by mlamouri@chromium.org, Apr 29 2016

Issue description

Audio usually has no frame as far as Blink is concerned unless the page asked for controls. If we request visibility for audio, we basically block autoplay.
 
Blocking: 487345
Cc: renganat...@chromium.org
Status: Started (was: Assigned)

Comment 3 Deleted

Project Member

Comment 4 by bugdroid1@chromium.org, May 4 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/994e975fd12ed9dff7ac3c3bf4e5fd34060caf4c

commit 994e975fd12ed9dff7ac3c3bf4e5fd34060caf4c
Author: mlamouri <mlamouri@chromium.org>
Date: Wed May 04 17:32:37 2016

Autoplay experiment: ignore viewport visibility requirements for audio.

Audio elements usually have no frame. Checking for viewport visibility
requirements would be fairly similar to simply not allowing autoplay or
forcing websites to switch to default controls.

BUG= 607918 
R=liberato@chromium.org

Review-Url: https://codereview.chromium.org/1931363002
Cr-Commit-Position: refs/heads/master@{#391552}

[modify] https://crrev.com/994e975fd12ed9dff7ac3c3bf4e5fd34060caf4c/third_party/WebKit/LayoutTests/media/audio-autoplay-experiment-modes-expected.txt
[modify] https://crrev.com/994e975fd12ed9dff7ac3c3bf4e5fd34060caf4c/third_party/WebKit/Source/core/html/AutoplayExperimentHelper.cpp
[modify] https://crrev.com/994e975fd12ed9dff7ac3c3bf4e5fd34060caf4c/third_party/WebKit/Source/core/html/AutoplayExperimentTest.cpp

Status: Fixed (was: Started)

Sign in to add a comment