Create an experiment to study whether too many bindings cause NAT failure

chrome/browser/chrome_browser_field_trials_desktop.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chrome_browser_field_trials_desktop.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
@@ skipping 27 matching lines @@
 void SetupLightSpeedTrials() {
   if (!variations::GetVariationParamValue("LightSpeed", "NoGpu").empty()) {
     base::CommandLine::ForCurrentProcess()->AppendSwitch(
         switches::kDisableGpu);
   }
 }
 
 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"];

[pthatcher2, 2015/10/29 14:13:13]

Why do we need 6 servers?

[guoweis_webrtc, 2015/10/29 21:45:29]

The parsing code will stop at the empty server so this is not an issue. Adding
more servers allow us to config it in finch if we do need that many servers in
the future.

 
   base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
       switches::kWebRtcStunProbeTrialParameter, cmd_param);
 #endif
 }
 
 }  // namespace
 
 void SetupDesktopFieldTrials(const base::CommandLine& parsed_command_line) {
   prerender::ConfigurePrerender(parsed_command_line);
   AutoLaunchChromeFieldTrial();
   SetupLightSpeedTrials();
   tracing::SetupBackgroundTracingFieldTrial();
   SetupStunProbeTrial();
 }
 
 }  // namespace chrome