Enable AllowBFCacheWhenClosedMediaStreamTrack by default
As our experiment goes well, we are planning to launch this feature.
This CL updates the feature to be enabled by default.
Bug: 1502395
Change-Id: I7db4af6f864fab2fd9a41fee74ebae569ec7122f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5410615
Commit-Queue: Mingyu Lei <leimy@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1281083}
diff --git a/testing/variations/fieldtrial_testing_config.json b/testing/variations/fieldtrial_testing_config.json
index db58d67..d3bb5e249 100644
--- a/testing/variations/fieldtrial_testing_config.json
+++ b/testing/variations/fieldtrial_testing_config.json
@@ -222,25 +222,6 @@
]
}
],
- "AllowBFCacheWhenClosedMediaStreamTrack": [
- {
- "platforms": [
- "android",
- "chromeos",
- "linux",
- "mac",
- "windows"
- ],
- "experiments": [
- {
- "name": "Enabled_20231219",
- "enable_features": [
- "AllowBFCacheWhenClosedMediaStreamTrack"
- ]
- }
- ]
- }
- ],
"AllowDatapipeDrainedAsBytesConsumerInBFCache": [
{
"platforms": [
diff --git a/third_party/blink/common/features.cc b/third_party/blink/common/features.cc
index d77af21..e1bacf37 100644
--- a/third_party/blink/common/features.cc
+++ b/third_party/blink/common/features.cc
@@ -56,10 +56,10 @@
base::FEATURE_DISABLED_BY_DEFAULT);
// Make MediaDevicesDispatcherHost and ended MediaStreamTrack not block BFCache.
-// See https://chrbug.com/1502395 for more details.
+// See https://crbug.com/1502395 for more details.
BASE_FEATURE(kAllowBFCacheWhenClosedMediaStreamTrack,
"AllowBFCacheWhenClosedMediaStreamTrack",
- base::FEATURE_DISABLED_BY_DEFAULT);
+ base::FEATURE_ENABLED_BY_DEFAULT);
// Block all MIDI access with the MIDI_SYSEX permission
BASE_FEATURE(kBlockMidiByDefault,
