Commit 402e5e20 authored by Ian Clelland's avatar Ian Clelland Committed by Commit Bot

Set DocumentPolicy status to experimental.

This also removes the virtual test suite, and the failing test
expectations from the experimental build.

The default values for the two initial features are also set to their
correct web-compatible defaults.

Bug: 993790
Change-Id: I308676b0c63524ad4a229a03940b928f95a384c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019464
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Reviewed-by: default avatarYaron Friedman <yfriedman@chromium.org>
Reviewed-by: default avatarJeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#735751}
parent 068a82af
...@@ -28,6 +28,9 @@ GetSwitchDependentFeatureOverrides(const base::CommandLine& command_line) { ...@@ -28,6 +28,9 @@ GetSwitchDependentFeatureOverrides(const base::CommandLine& command_line) {
{switches::kEnableExperimentalWebPlatformFeatures, {switches::kEnableExperimentalWebPlatformFeatures,
std::cref(network::features::kCrossOriginIsolation), std::cref(network::features::kCrossOriginIsolation),
base::FeatureList::OVERRIDE_ENABLE_FEATURE}, base::FeatureList::OVERRIDE_ENABLE_FEATURE},
{switches::kEnableExperimentalWebPlatformFeatures,
std::cref(features::kDocumentPolicy),
base::FeatureList::OVERRIDE_ENABLE_FEATURE},
}; };
// TODO(chlily): There are currently a few places where, to check if some // TODO(chlily): There are currently a few places where, to check if some
......
...@@ -280,9 +280,10 @@ std::unique_ptr<DocumentPolicy> DocumentPolicy::CreateWithHeaderPolicy( ...@@ -280,9 +280,10 @@ std::unique_ptr<DocumentPolicy> DocumentPolicy::CreateWithHeaderPolicy(
// configuration json5 file. // configuration json5 file.
const DocumentPolicy::FeatureState& DocumentPolicy::GetFeatureDefaults() { const DocumentPolicy::FeatureState& DocumentPolicy::GetFeatureDefaults() {
static base::NoDestructor<FeatureState> default_feature_list( static base::NoDestructor<FeatureState> default_feature_list(
{{mojom::FeaturePolicyFeature::kFontDisplay, PolicyValue(false)}, {{mojom::FeaturePolicyFeature::kFontDisplay, PolicyValue(true)},
{mojom::FeaturePolicyFeature::kUnoptimizedLosslessImages, {mojom::FeaturePolicyFeature::kUnoptimizedLosslessImages,
PolicyValue(2.0f)}}); PolicyValue::CreateMaxPolicyValue(
mojom::PolicyValueType::kDecDouble)}});
return *default_feature_list; return *default_feature_list;
} }
......
...@@ -608,6 +608,7 @@ ...@@ -608,6 +608,7 @@
}, },
{ {
name: "DocumentPolicy", name: "DocumentPolicy",
status: "experimental",
}, },
{ {
name: "DocumentWrite", name: "DocumentWrite",
......
...@@ -6153,12 +6153,7 @@ crbug.com/1006759 http/tests/devtools/modules-load-network.js [ Pass Failure ] ...@@ -6153,12 +6153,7 @@ crbug.com/1006759 http/tests/devtools/modules-load-network.js [ Pass Failure ]
crbug.com/1006759 http/tests/devtools/modules-load-source.js [ Pass Failure ] crbug.com/1006759 http/tests/devtools/modules-load-source.js [ Pass Failure ]
# Failing document policy tests # Failing document policy tests
crbug.com/993790 external/wpt/document-policy/required-policy/document-policy.html [ Failure ]
crbug.com/993790 external/wpt/document-policy/required-policy/no-document-policy.html [ Failure ]
crbug.com/993790 external/wpt/document-policy/required-policy/required-document-policy-nested.html [ Failure ]
crbug.com/993790 external/wpt/document-policy/required-policy/required-document-policy.html [ Failure ]
crbug.com/993790 external/wpt/document-policy/required-policy/separate-document-policies.html [ Failure ] crbug.com/993790 external/wpt/document-policy/required-policy/separate-document-policies.html [ Failure ]
crbug.com/993790 virtual/document-policy/external/wpt/document-policy/required-policy/separate-document-policies.html [ Failure ]
# Temporary suppression to allow devtools-frontend changes # Temporary suppression to allow devtools-frontend changes
crbug.com/1029489 http/tests/devtools/elements/elements-linkify-attributes.js [ Pass Failure ] crbug.com/1029489 http/tests/devtools/elements/elements-linkify-attributes.js [ Pass Failure ]
......
...@@ -701,11 +701,6 @@ ...@@ -701,11 +701,6 @@
"bases": ["external/wpt/fetch/http-cache"], "bases": ["external/wpt/fetch/http-cache"],
"args": ["--disable-features=SplitCacheByNetworkIsolationKey"] "args": ["--disable-features=SplitCacheByNetworkIsolationKey"]
}, },
{
"prefix": "document-policy",
"bases": ["external/wpt/document-policy"],
"args": ["--enable-features=DocumentPolicy"]
},
{ {
"prefix": "web-components-v0-disabled", "prefix": "web-components-v0-disabled",
"bases": ["external/wpt/dom", "bases": ["external/wpt/dom",
......
This test contains expectations for document policy web platform tests when
run with document policy enabled.
...@@ -659,6 +659,7 @@ html element iframe ...@@ -659,6 +659,7 @@ html element iframe
property marginHeight property marginHeight
property marginWidth property marginWidth
property name property name
property policy
property referrerPolicy property referrerPolicy
property sandbox property sandbox
property scrolling property scrolling
......
...@@ -3387,6 +3387,7 @@ interface HTMLIFrameElement : HTMLElement ...@@ -3387,6 +3387,7 @@ interface HTMLIFrameElement : HTMLElement
getter marginHeight getter marginHeight
getter marginWidth getter marginWidth
getter name getter name
getter policy
getter referrerPolicy getter referrerPolicy
getter sandbox getter sandbox
getter scrolling getter scrolling
...@@ -3408,6 +3409,7 @@ interface HTMLIFrameElement : HTMLElement ...@@ -3408,6 +3409,7 @@ interface HTMLIFrameElement : HTMLElement
setter marginHeight setter marginHeight
setter marginWidth setter marginWidth
setter name setter name
setter policy
setter referrerPolicy setter referrerPolicy
setter sandbox setter sandbox
setter scrolling setter scrolling
......
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