Commit 63bcf45b authored by Tarun Bansal's avatar Tarun Bansal Committed by Commit Bot

Enable signal strength based ECT capping

Around 30-40% more previews with no significant change in opt-out rate.
See the bug for details.

Change-Id: I299dc5f15e32033f2f463d3b49fc3f30e157593d
Bug: 924160
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1552573Reviewed-by: default avatarRyan Sturm <ryansturm@chromium.org>
Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#648303}
parent dbd6ce1b
......@@ -501,7 +501,7 @@ NetworkQualityEstimatorParams::NetworkQualityEstimatorParams(
GetStringValueForVariationParamWithDefaultValue(
params_,
"cap_ect_based_on_signal_strength",
"false") == "true"),
"true") != "false"),
upper_bound_typical_kbps_multiplier_(
GetDoubleValueForVariationParamWithDefaultValue(
params_,
......
......@@ -3010,28 +3010,6 @@
]
}
],
"NetworkQualityEstimator": [
{
"platforms": [
"android",
"windows",
"mac",
"chromeos",
"linux"
],
"experiments": [
{
"name": "Enabled9",
"params": {
"cap_ect_based_on_signal_strength": "true"
},
"enable_features": [
"NetworkQualityEstimator"
]
}
]
}
],
"NewExtensionUpdaterService": [
{
"platforms": [
......
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