Adds ability to disable connection pooling in QUIC.

chrome/browser/io_thread_unittest.cc

 // Copyright 2014 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 "base/command_line.h"
 #include "chrome/browser/io_thread.h"
 #include "net/http/http_network_session.h"
 #include "net/http/http_server_properties_impl.h"
 #include "net/quic/quic_protocol.h"
 #include "testing/gmock/include/gmock/gmock.h"
@@ skipping 115 matching lines @@
   net::HttpNetworkSession::Params default_params;
   net::HttpNetworkSession::Params params;
   InitializeNetworkSessionParams(&params);
   EXPECT_TRUE(params.enable_quic);
   EXPECT_FALSE(params.enable_quic_time_based_loss_detection);
   EXPECT_EQ(1350u, params.quic_max_packet_length);
   EXPECT_EQ(default_params.quic_supported_versions,
             params.quic_supported_versions);
   EXPECT_EQ(net::QuicTagVector(), params.quic_connection_options);
   EXPECT_FALSE(params.quic_always_require_handshake_confirmation);
+  EXPECT_FALSE(params.quic_disable_connection_pooling);
 }
 
 TEST_F(IOThreadTest, EnableQuicFromCommandLine) {
   base::CommandLine::StringVector args;
   command_line_.AppendSwitch("enable-quic");
 
   ConfigureQuicGlobals();
   net::HttpNetworkSession::Params params;
   InitializeNetworkSessionParams(&params);
   EXPECT_TRUE(params.enable_quic);
@@ skipping 178 matching lines @@
 TEST_F(IOThreadTest,
        QuicAlwaysRequireHandshakeConfirmationFromFieldTrialParams) {
   field_trial_group_ = "Enabled";
   field_trial_params_["always_require_handshake_confirmation"] = "true";
   ConfigureQuicGlobals();
   net::HttpNetworkSession::Params params;
   InitializeNetworkSessionParams(&params);
   EXPECT_TRUE(params.quic_always_require_handshake_confirmation);
 }
 
+TEST_F(IOThreadTest,
+       QuicDisableConnectionPoolingFromFieldTrialParams) {
+  field_trial_group_ = "Enabled";
+  field_trial_params_["DisableConnectionPooling"] = "true";

[Ryan Hamilton, 2014/09/08 17:39:15]

nit: I think we've been using hacker_case for the other params. Can you switch
this do disable_connection_pooling?

[Jana, 2014/09/08 18:49:41]

Ah sorry -- this was an oversight. Done.

+  ConfigureQuicGlobals();
+  net::HttpNetworkSession::Params params;
+  InitializeNetworkSessionParams(&params);
+  EXPECT_TRUE(params.quic_disable_connection_pooling);
+}
+
 }  // namespace test