Commit ceb19533 authored by Xianzhu Wang's avatar Xianzhu Wang Committed by Commit Bot

Revert "Add blink::features::CompositeSVG"

This reverts commit 4562b3f8.

Reason for revert: This prevents the blink feature from being tested
for web tests. Will land crrev.com/c/2369263 first. We may not reland
this CL if we decide not to go through field trial.

Original change's description:
> Add blink::features::CompositeSVG
> 
> It corresponds to blink::RuntimeEnabledFeatures::CompositeSVGEnabled().
> This is to allow perf tests with CompositeSVG enabled by default with a
> testing variation.
> 
> Bug: 1101002
> Change-Id: If9a92b9eb56d0b242d8a46e397df71faa01577e6
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2373398
> Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
> Reviewed-by: Philip Rogers <pdr@chromium.org>
> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#801382}

TBR=wangxianzhu@chromium.org,pdr@chromium.org,chrishtr@chromium.org

Change-Id: I8546a075b9d850a53e9117d12731ee9fe1ae3d6f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1101002
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2375996Reviewed-by: default avatarXianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#801623}
parent 8f2b175c
...@@ -341,7 +341,6 @@ void SetRuntimeFeaturesFromChromiumFeatures() { ...@@ -341,7 +341,6 @@ void SetRuntimeFeaturesFromChromiumFeatures() {
{wf::EnableMediaFeeds, media::kMediaFeeds, kUseFeatureState}, {wf::EnableMediaFeeds, media::kMediaFeeds, kUseFeatureState},
{wf::EnableRestrictGamepadAccess, features::kRestrictGamepadAccess, {wf::EnableRestrictGamepadAccess, features::kRestrictGamepadAccess,
kEnableOnly}, kEnableOnly},
{wf::EnableCompositeSVG, blink::features::kCompositeSVG, kUseFeatureState},
{wf::EnableCompositingOptimizations, {wf::EnableCompositingOptimizations,
blink::features::kCompositingOptimizations, kUseFeatureState}, blink::features::kCompositingOptimizations, kUseFeatureState},
{wf::EnableConversionMeasurementInfraSupport, {wf::EnableConversionMeasurementInfraSupport,
......
...@@ -692,11 +692,6 @@ bool IsParkableStringsToDiskEnabled() { ...@@ -692,11 +692,6 @@ bool IsParkableStringsToDiskEnabled() {
const base::Feature kCrOSAutoSelect{"CrOSAutoSelect", const base::Feature kCrOSAutoSelect{"CrOSAutoSelect",
base::FEATURE_DISABLED_BY_DEFAULT}; base::FEATURE_DISABLED_BY_DEFAULT};
// Enables compositing of eligible SVG elements to improve animation
// performance. See crbug.com/1101002.
const base::Feature kCompositeSVG{"CompositeSVG",
base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kCompositingOptimizations{ const base::Feature kCompositingOptimizations{
"CompositingOptimizations", base::FEATURE_DISABLED_BY_DEFAULT}; "CompositingOptimizations", base::FEATURE_DISABLED_BY_DEFAULT};
......
...@@ -277,7 +277,6 @@ BLINK_COMMON_EXPORT bool IsParkableStringsToDiskEnabled(); ...@@ -277,7 +277,6 @@ BLINK_COMMON_EXPORT bool IsParkableStringsToDiskEnabled();
BLINK_COMMON_EXPORT extern const base::Feature kCrOSAutoSelect; BLINK_COMMON_EXPORT extern const base::Feature kCrOSAutoSelect;
BLINK_COMMON_EXPORT extern const base::Feature kCompositeSVG;
BLINK_COMMON_EXPORT extern const base::Feature kCompositingOptimizations; BLINK_COMMON_EXPORT extern const base::Feature kCompositingOptimizations;
BLINK_COMMON_EXPORT extern const base::Feature kReducedReferrerGranularity; BLINK_COMMON_EXPORT extern const base::Feature kReducedReferrerGranularity;
......
...@@ -253,7 +253,6 @@ class WebRuntimeFeatures { ...@@ -253,7 +253,6 @@ class WebRuntimeFeatures {
BLINK_PLATFORM_EXPORT static void EnableConversionMeasurementInfraSupport( BLINK_PLATFORM_EXPORT static void EnableConversionMeasurementInfraSupport(
bool); bool);
BLINK_PLATFORM_EXPORT static void EnableCompositeSVG(bool);
BLINK_PLATFORM_EXPORT static void EnableCompositingOptimizations(bool); BLINK_PLATFORM_EXPORT static void EnableCompositingOptimizations(bool);
private: private:
......
...@@ -717,10 +717,6 @@ void WebRuntimeFeatures::EnableRestrictGamepadAccess(bool enable) { ...@@ -717,10 +717,6 @@ void WebRuntimeFeatures::EnableRestrictGamepadAccess(bool enable) {
RuntimeEnabledFeatures::SetRestrictGamepadAccessEnabled(enable); RuntimeEnabledFeatures::SetRestrictGamepadAccessEnabled(enable);
} }
void WebRuntimeFeatures::EnableCompositeSVG(bool enable) {
RuntimeEnabledFeatures::SetCompositeSVGEnabled(enable);
}
void WebRuntimeFeatures::EnableCompositingOptimizations(bool enable) { void WebRuntimeFeatures::EnableCompositingOptimizations(bool enable) {
RuntimeEnabledFeatures::SetCompositingOptimizationsEnabled(enable); RuntimeEnabledFeatures::SetCompositingOptimizationsEnabled(enable);
} }
......
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