Enable V8DelayMemoryReducer by default.
We experiment with this feature on stable (30 delay) with satisfactory
results go/v8delaymemoryreducer-results-2022. Following that, we
included the feature in a "CombinedPowerExperiment" which also has
satisfactory results. In preparation for launching
"CombinedPowerExperiment", we enable V8DelayMemoryReducer by default.
Bug: 1330940
Change-Id: I29a0d4737e449a96c7aae0e4f890076429d6286c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4095261
Reviewed-by: Michael Lippautz <[email protected]>
Auto-Submit: Francois Pierre Doray <[email protected]>
Commit-Queue: Francois Pierre Doray <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1082502}
diff --git a/gin/gin_features.cc b/gin/gin_features.cc
index eec07b2..6a90e38 100644
--- a/gin/gin_features.cc
+++ b/gin/gin_features.cc
@@ -146,9 +146,9 @@
BASE_FEATURE(kV8DelayMemoryReducer,
"V8DelayMemoryReducer",
- base::FEATURE_DISABLED_BY_DEFAULT);
+ base::FEATURE_ENABLED_BY_DEFAULT);
const base::FeatureParam<base::TimeDelta> kV8MemoryReducerStartDelay{
- &kV8DelayMemoryReducer, "delay", base::Seconds(8)};
+ &kV8DelayMemoryReducer, "delay", base::Seconds(30)};
// JavaScript language features.
