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

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: update UMA name in code to match with histograms.xml Created 5 years, 1 month 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 | content/renderer/media/webrtc/peer_connection_dependency_factory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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);
« no previous file with comments | « no previous file | content/renderer/media/webrtc/peer_connection_dependency_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698