QUIC - added disable_delay_tcp_race field trial param to disable delaying of racing TCP connection

chrome/browser/io_thread.cc

Index: chrome/browser/io_thread.cc
diff --git a/chrome/browser/io_thread.cc b/chrome/browser/io_thread.cc
index 5d6949b978786da0d78f4bfb6481901a5a58e75a..7a17d92217dedd6ebee99d0949d7daf065a32a37 100644
--- a/chrome/browser/io_thread.cc
+++ b/chrome/browser/io_thread.cc
@@ -1451,8 +1451,13 @@ int IOThread::NetworkSessionConfigurator::GetQuicSocketReceiveBufferSize(
 // static
 bool IOThread::NetworkSessionConfigurator::ShouldQuicDelayTcpRace(
     const VariationParameters& quic_trial_params) {
-  return base::LowerCaseEqualsASCII(
-      GetVariationParam(quic_trial_params, "delay_tcp_race"), "true");
+  std::string delay_tcp_race = GetVariationParam(quic_trial_params,

[ramant (doing other things), 2016/05/20 00:00:49]

Hi Ryan,
  Another option is to rename this parameter to "disable_delay_tcp_race" and
change all the code accordingly. What do you think?

thanks
raman

[Ryan Hamilton, 2016/05/20 03:17:27]

Yeah, it seems like that would be a better idea.

[ramant (doing other things), 2016/05/24 16:52:02]

Done.

+                                                 "delay_tcp_race");
+  // If "delay_tcp_race" parameter is not set, then default it to the default
+  // value which is true. Otherwise, if it is set, then return the value set by
+  // the parameter.
+  return delay_tcp_race.empty() || base::LowerCaseEqualsASCII(delay_tcp_race,
+                                                              "true");
 }
 
 // static