Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(516)

Unified Diff: content/browser/memory/memory_state_updater.cc

Issue 2717953004: Rename memory coordinator's variation parameters part 1 (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/memory/memory_state_updater.cc
diff --git a/content/browser/memory/memory_state_updater.cc b/content/browser/memory/memory_state_updater.cc
index fb75dbd2016aecfef0b80fb4da54a4826a0c76f2..a432fb3745ae57e861d369f71856e88e37f5fbd6 100644
--- a/content/browser/memory/memory_state_updater.cc
+++ b/content/browser/memory/memory_state_updater.cc
@@ -147,16 +147,25 @@ void MemoryStateUpdater::InitializeParameters() {
static constexpr char kMemoryCoordinatorV0Trial[] = "MemoryCoordinatorV0";
std::map<std::string, std::string> params;
variations::GetVariationParams(kMemoryCoordinatorV0Trial, &params);
+ // TODO(bashi): Renaming (throttled -> warning, suspended -> critical) is
+ // ongoing. Get variation parameters from both until server-side change
+ // is done. crbug.com/696844
SetIntVariationParameter(params, "expected_renderer_size",
&expected_renderer_size_);
SetIntVariationParameter(params, "new_renderers_until_throttled",
&new_renderers_until_throttled_);
+ SetIntVariationParameter(params, "new_renderers_until_warning",
+ &new_renderers_until_throttled_);
SetIntVariationParameter(params, "new_renderers_until_suspended",
&new_renderers_until_suspended_);
+ SetIntVariationParameter(params, "new_renderers_until_critical",
+ &new_renderers_until_suspended_);
SetIntVariationParameter(params, "new_renderers_back_to_normal",
&new_renderers_back_to_normal_);
SetIntVariationParameter(params, "new_renderers_back_to_throttled",
&new_renderers_back_to_throttled_);
+ SetIntVariationParameter(params, "new_renderers_back_to_warning",
+ &new_renderers_back_to_throttled_);
SetSecondsVariationParameter(params, "minimum_transition_period",
&minimum_transition_period_);
SetSecondsVariationParameter(params, "monitoring_interval",
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698