Commit 6c56671c authored by Dave Tapuska's avatar Dave Tapuska Committed by Commit Bot

Remove long shipped CSS Environment variable setting.

The feature first shipped two stable releases ago in M69.

Bug=848890

Change-Id: Id969cf348d9523ec50803d7feadec7071b652730
Reviewed-on: https://chromium-review.googlesource.com/c/1323770Reviewed-by: default avatarRune Lillesveen <futhark@chromium.org>
Reviewed-by: default avatarBecca Hughes <beccahughes@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606161}
parent cb04d0bc
...@@ -36,8 +36,6 @@ bool ClassifyBlock(CSSParserTokenRange range, ...@@ -36,8 +36,6 @@ bool ClassifyBlock(CSSParserTokenRange range,
has_references = true; has_references = true;
continue; continue;
case CSSValueEnv: case CSSValueEnv:
if (!RuntimeEnabledFeatures::CSSEnvironmentVariablesEnabled())
return false;
if (!IsValidEnvVariableReference(range.ConsumeBlock(), if (!IsValidEnvVariableReference(range.ConsumeBlock(),
skip_variables)) skip_variables))
return false; // Invalid reference. return false; // Invalid reference.
......
...@@ -40,7 +40,6 @@ class CSSVariableResolverTest : public PageTestBase { ...@@ -40,7 +40,6 @@ class CSSVariableResolverTest : public PageTestBase {
void SetUp() override { void SetUp() override {
PageTestBase::SetUp(); PageTestBase::SetUp();
RuntimeEnabledFeatures::SetCSSEnvironmentVariablesEnabled(true);
GetStyleEngine().EnsureEnvironmentVariables().SetVariable("test", "red"); GetStyleEngine().EnsureEnvironmentVariables().SetVariable("test", "red");
} }
......
...@@ -41,8 +41,6 @@ class StyleEnvironmentVariablesTest : public PageTestBase { ...@@ -41,8 +41,6 @@ class StyleEnvironmentVariablesTest : public PageTestBase {
void SetUp() override { void SetUp() override {
PageTestBase::SetUp(); PageTestBase::SetUp();
RuntimeEnabledFeatures::SetCSSEnvironmentVariablesEnabled(true);
// Needed for RecordUseCounter_IgnoreMediaControls. // Needed for RecordUseCounter_IgnoreMediaControls.
RuntimeEnabledFeatures::SetModernMediaControlsEnabled(true); RuntimeEnabledFeatures::SetModernMediaControlsEnabled(true);
} }
......
...@@ -249,10 +249,6 @@ ...@@ -249,10 +249,6 @@
name: "CSSBackdropFilter", name: "CSSBackdropFilter",
status: "experimental", status: "experimental",
}, },
{
name: "CSSEnvironmentVariables",
status: "stable",
},
{ {
name: "CSSFocusVisible", name: "CSSFocusVisible",
status: "experimental", status: "experimental",
......
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