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

Issue 618083 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android , All
Pri: 1
Type: Bug

Blocking:
issue 617592



Sign in to add a comment

Show autoplay settings if autoplay muted video flag is on

Project Member Reported by mlamouri@chromium.org, Jun 7 2016

Issue description

We currently show the settings base on the experiments. We want to go ahead of the experiments here so we should change the rule. It should be a fairly trivial change.
 
Cc: finnur@chromium.org
finnur@, by any chance, could you point to me where the check is made so I could do this change when the flag has landed? You can also own this and I can ping you with the flag name when it's ready. As you prefer :)
Yes. Thanks! :)
Cc: mlamouri@chromium.org
Labels: OS-Android
Owner: avayvod@chromium.org
Status: Started (was: Available)
Started looking into this.
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 10 2016

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

commit 5d05e65d8ab098888e1e87cd51f93496e93a8398
Author: avayvod <avayvod@chromium.org>
Date: Fri Jun 10 16:04:46 2016

[Android, Media] Enable autoplay settings for muted autoplay, wired them up to the media element.

Use chrome://flags/#enable-autoplay-muted-videos to enable the autoplay site setting.

Check the setting before allowing muted videos to autoplay.

BUG= 618083 , 618081 
TEST=Manual

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

[modify] https://crrev.com/5d05e65d8ab098888e1e87cd51f93496e93a8398/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SiteSettingsPreferences.java
[modify] https://crrev.com/5d05e65d8ab098888e1e87cd51f93496e93a8398/third_party/WebKit/LayoutTests/media/autoplay-muted.html
[modify] https://crrev.com/5d05e65d8ab098888e1e87cd51f93496e93a8398/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/5d05e65d8ab098888e1e87cd51f93496e93a8398/third_party/WebKit/Source/core/html/HTMLMediaElement.h

Status: Fixed (was: Started)
Project Member

Comment 7 by bugdroid1@chromium.org, Jun 15 2016

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

commit 5d05e65d8ab098888e1e87cd51f93496e93a8398
Author: avayvod <avayvod@chromium.org>
Date: Fri Jun 10 16:04:46 2016

[Android, Media] Enable autoplay settings for muted autoplay, wired them up to the media element.

Use chrome://flags/#enable-autoplay-muted-videos to enable the autoplay site setting.

Check the setting before allowing muted videos to autoplay.

BUG= 618083 , 618081 
TEST=Manual

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

[modify] https://crrev.com/5d05e65d8ab098888e1e87cd51f93496e93a8398/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SiteSettingsPreferences.java
[modify] https://crrev.com/5d05e65d8ab098888e1e87cd51f93496e93a8398/third_party/WebKit/LayoutTests/media/autoplay-muted.html
[modify] https://crrev.com/5d05e65d8ab098888e1e87cd51f93496e93a8398/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/5d05e65d8ab098888e1e87cd51f93496e93a8398/third_party/WebKit/Source/core/html/HTMLMediaElement.h

Status: Verified (was: Fixed)
Verified in 53.0.2780.0 build.
Now Media option is displayed under settings after enable the flag (autoplay-muted videos)in about://flags page

Sign in to add a comment