Commit 2719d34c authored by Eric Willigers's avatar Eric Willigers Committed by Commit Bot

Retire FeaturePolicyAutoplayFeature flag.

Shipped in M64, this feature allows developers to selectively enable
and disable use of autoplay through the feature policy HTTP header
or the <iframe> "allow" attribute.

https://chromium-review.googlesource.com/831520
https://www.chromestatus.com/features/5100524789563392

BUG=788390

Change-Id: I0f4346bc75dc922c8004f0008921d68a5a3beede
Reviewed-on: https://chromium-review.googlesource.com/1166614
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Reviewed-by: default avatarBecca Hughes <beccahughes@chromium.org>
Reviewed-by: default avatarJochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581968}
parent 3328faef
...@@ -113,8 +113,7 @@ bool AutoplayPolicy::IsDocumentAllowedToPlay(const Document& document) { ...@@ -113,8 +113,7 @@ bool AutoplayPolicy::IsDocumentAllowedToPlay(const Document& document) {
return true; return true;
} }
if (!RuntimeEnabledFeatures::FeaturePolicyAutoplayFeatureEnabled() || if (!frame->IsFeatureEnabled(mojom::FeaturePolicyFeature::kAutoplay)) {
!frame->IsFeatureEnabled(mojom::FeaturePolicyFeature::kAutoplay)) {
return false; return false;
} }
} }
......
...@@ -210,6 +210,8 @@ void AllowFeatureEverywhere(mojom::FeaturePolicyFeature feature, ...@@ -210,6 +210,8 @@ void AllowFeatureEverywhere(mojom::FeaturePolicyFeature feature,
const FeatureNameMap& GetDefaultFeatureNameMap() { const FeatureNameMap& GetDefaultFeatureNameMap() {
DEFINE_STATIC_LOCAL(FeatureNameMap, default_feature_name_map, ()); DEFINE_STATIC_LOCAL(FeatureNameMap, default_feature_name_map, ());
if (default_feature_name_map.IsEmpty()) { if (default_feature_name_map.IsEmpty()) {
default_feature_name_map.Set("autoplay",
mojom::FeaturePolicyFeature::kAutoplay);
default_feature_name_map.Set("camera", default_feature_name_map.Set("camera",
mojom::FeaturePolicyFeature::kCamera); mojom::FeaturePolicyFeature::kCamera);
default_feature_name_map.Set("encrypted-media", default_feature_name_map.Set("encrypted-media",
...@@ -249,10 +251,6 @@ const FeatureNameMap& GetDefaultFeatureNameMap() { ...@@ -249,10 +251,6 @@ const FeatureNameMap& GetDefaultFeatureNameMap() {
default_feature_name_map.Set("sync-script", default_feature_name_map.Set("sync-script",
mojom::FeaturePolicyFeature::kSyncScript); mojom::FeaturePolicyFeature::kSyncScript);
} }
if (RuntimeEnabledFeatures::FeaturePolicyAutoplayFeatureEnabled()) {
default_feature_name_map.Set("autoplay",
mojom::FeaturePolicyFeature::kAutoplay);
}
if (RuntimeEnabledFeatures::PaymentRequestEnabled()) { if (RuntimeEnabledFeatures::PaymentRequestEnabled()) {
default_feature_name_map.Set("payment", default_feature_name_map.Set("payment",
mojom::FeaturePolicyFeature::kPayment); mojom::FeaturePolicyFeature::kPayment);
......
...@@ -455,10 +455,6 @@ ...@@ -455,10 +455,6 @@
{ {
name: "FastMobileScrolling", name: "FastMobileScrolling",
}, },
{
name: "FeaturePolicyAutoplayFeature",
status: "stable"
},
{ {
name: "FeaturePolicyJavaScriptInterface", name: "FeaturePolicyJavaScriptInterface",
origin_trial_feature_name: "FeaturePolicyJSAPI", origin_trial_feature_name: "FeaturePolicyJSAPI",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment