Commit 03df4919 authored by Xiaocheng Hu's avatar Xiaocheng Hu Committed by Commit Bot

Revert "Enable full testing of CSSReducedFontLoadingLayoutInvalidations"

This reverts commit 5aba5002.

Reason for revert: Causing crbug.com/1092431

Let's stop messing up the perf bots while debugging.

Original change's description:
> Enable full testing of CSSReducedFontLoadingLayoutInvalidations
> 
> This is preparation for enabling finch experiments of the feature on M85
> 
> Note: though M85 hasn't reached Beta, we'd like to have it covered by
> test bots first to catch any breakage. We'd also like it to be tested
> by perf bots, since this "feature" is a performance optimization.
> 
> Bug: 441925
> Change-Id: I3ef98ffcfa960381efd034cd1cd63e55fbda4b01
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2231269
> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
> Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
> Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#775847}

TBR=asvitkine@chromium.org,chrishtr@chromium.org,xiaochengh@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 441925, 1092431
Change-Id: Ife4aa822dda70d88249306b63e30d3d39df7a560
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2242383
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Reviewed-by: default avatarXiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#778044}
parent 44f63222
...@@ -356,9 +356,6 @@ void SetRuntimeFeaturesFromChromiumFeatures() { ...@@ -356,9 +356,6 @@ void SetRuntimeFeaturesFromChromiumFeatures() {
{"CSSReducedFontLoadingInvalidations", {"CSSReducedFontLoadingInvalidations",
blink::features::kCSSReducedFontLoadingInvalidations, blink::features::kCSSReducedFontLoadingInvalidations,
kUseFeatureState}, kUseFeatureState},
{"CSSReducedFontLoadingLayoutInvalidations",
blink::features::kCSSReducedFontLoadingLayoutInvalidations,
kUseFeatureState},
{"CSSMatchedPropertiesCacheDependencies", {"CSSMatchedPropertiesCacheDependencies",
blink::features::kCSSMatchedPropertiesCacheDependencies, blink::features::kCSSMatchedPropertiesCacheDependencies,
kUseFeatureState}, kUseFeatureState},
......
...@@ -1271,27 +1271,6 @@ ...@@ -1271,27 +1271,6 @@
] ]
} }
], ],
"CSSReducedFontLoadingLayoutInvalidations": [
{
"platforms": [
"android",
"android_weblayer",
"android_webview",
"chromeos",
"linux",
"mac",
"windows"
],
"experiments": [
{
"name": "Enabled",
"enable_features": [
"CSSReducedFontLoadingLayoutInvalidations"
]
}
]
}
],
"CacheStorageEagerReading": [ "CacheStorageEagerReading": [
{ {
"platforms": [ "platforms": [
......
...@@ -464,9 +464,6 @@ const base::Feature kDawn2dCanvas{"Dawn2dCanvas", ...@@ -464,9 +464,6 @@ const base::Feature kDawn2dCanvas{"Dawn2dCanvas",
const base::Feature kCSSReducedFontLoadingInvalidations{ const base::Feature kCSSReducedFontLoadingInvalidations{
"CSSReducedFontLoadingInvalidations", base::FEATURE_DISABLED_BY_DEFAULT}; "CSSReducedFontLoadingInvalidations", base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kCSSReducedFontLoadingLayoutInvalidations{
"CSSReducedFontLoadingLayoutInvalidations",
base::FEATURE_DISABLED_BY_DEFAULT};
// When enabled, frees up CachedMetadata after consumption by script resources // When enabled, frees up CachedMetadata after consumption by script resources
// and modules. Needed for the experiment in http://crbug.com/1045052. // and modules. Needed for the experiment in http://crbug.com/1045052.
......
...@@ -147,8 +147,6 @@ BLINK_COMMON_EXPORT extern const base::Feature kDawn2dCanvas; ...@@ -147,8 +147,6 @@ BLINK_COMMON_EXPORT extern const base::Feature kDawn2dCanvas;
BLINK_COMMON_EXPORT extern const base::Feature BLINK_COMMON_EXPORT extern const base::Feature
kCSSReducedFontLoadingInvalidations; kCSSReducedFontLoadingInvalidations;
BLINK_COMMON_EXPORT extern const base::Feature
kCSSReducedFontLoadingLayoutInvalidations;
BLINK_COMMON_EXPORT extern const base::Feature kDiscardCodeCacheAfterFirstUse; BLINK_COMMON_EXPORT extern const base::Feature kDiscardCodeCacheAfterFirstUse;
......
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