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

Unified Diff: chrome/browser/chrome_browser_field_trials_desktop.cc

Issue 1417663004: Create an experiment to study whether too many bindings cause NAT failure (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address sergeyu's comments. Created 5 years, 2 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
Index: chrome/browser/chrome_browser_field_trials_desktop.cc
diff --git a/chrome/browser/chrome_browser_field_trials_desktop.cc b/chrome/browser/chrome_browser_field_trials_desktop.cc
index 50bde4d5d5571148a14f1dc63ab8fba2514f4a2d..4e6ac084aab632e7b0130ce43bf34c3b3c3b55a9 100644
--- a/chrome/browser/chrome_browser_field_trials_desktop.cc
+++ b/chrome/browser/chrome_browser_field_trials_desktop.cc
@@ -45,16 +45,18 @@ void SetupLightSpeedTrials() {
void SetupStunProbeTrial() {
#if defined(ENABLE_WEBRTC)
std::map<std::string, std::string> params;
- if (!variations::GetVariationParams("StunProbeTrial", &params))
+ if (!variations::GetVariationParams("StunProbeTrial2", &params))
return;
// The parameter, used by StartStunFieldTrial, should have the following
- // format: "request_per_ip/interval/sharedsocket/server1:port/server2:port/
- // server3:port/"
+ // format: "request_per_ip/interval/sharedsocket/batch_size/total_batches/
+ // server1:port/server2:port/server3:port/"
std::string cmd_param = params["request_per_ip"] + "/" + params["interval"] +
"/" + params["sharedsocket"] + "/" +
- params["server1"] + "/" + params["server2"] + "/" +
- params["server3"] + "/";
+ params["batch_size"] + "/" + params["total_batches"] +
+ "/" + params["server1"] + "/" + params["server2"] +
+ "/" + params["server3"] + "/" + params["server4"] +
+ "/" + params["server5"] + "/" + params["server6"];
base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
switches::kWebRtcStunProbeTrialParameter, cmd_param);

Powered by Google App Engine
This is Rietveld 408576698