[Cronet] Add QUIC experimental params

components/cronet/url_request_context_config_unittest.cc

+// Copyright 2015 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 "components/cronet/url_request_context_config.h"
+
+#include "net/http/http_network_session.h"
+#include "net/proxy/proxy_config.h"
+#include "net/proxy/proxy_config_service_fixed.h"
+#include "net/url_request/url_request_context.h"
+#include "net/url_request/url_request_context_builder.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace cronet {
+
+TEST(URLRequestContextConfigTest, SetQuicExperimentalOptions) {
+  URLRequestContextConfig config;
+
+  std::string args =
+      "{\"QUIC_HINTS\":[{\"QUIC_HINT_ALT_PORT\":6121,\"QUIC_HINT_PORT\":6121,"
+      "\"QUIC_HINT_HOST\":\"test.example.com\"}],"
+      "\"HTTP_CACHE\":\"HTTP_CACHE_DISK\",\"ENABLE_SDCH\":false,"
+      "\"ENABLE_LEGACY_MODE\":false,\"HTTP_CACHE_MAX_SIZE\":1024000,"
+      "\"NATIVE_LIBRARY_NAME\":\"cronet_tests\",\"USER_AGENT\":\"fake agent\","
+      "\"STORAGE_PATH\":"
+      "\"\\/data\\/data\\/org.chromium.net\\/app_cronet_test\\/test_storage\","
+      "\"ENABLE_SPDY\":true,"
+      "\"ENABLE_QUIC\":true,\"LOAD_DISABLE_CACHE\":true,"
+      "\"EXPERIMENTAL_OPTIONS\":"
+      "\"{\\\"QUIC\\\":{\\\"store_server_configs_in_properties\\\":true,"
+      "\\\"delay_tcp_race\\\":true,"
+      "\\\"connection_options\\\":\\\"TIME,TBBR,REJ\\\"}}\"}";
+  config.LoadFromJSON(args);
+  net::URLRequestContextBuilder builder;
+  config.ConfigureURLRequestContextBuilder(&builder);
+  // Set a ProxyConfigService to advoid DCHECK failure when building.

[ramant (doing other things), 2015/11/18 20:22:08]

nit: advoid -> avoid.

[xunjieli, 2015/11/18 20:58:44]

Done.

+  builder.set_proxy_config_service(make_scoped_ptr(
+      new net::ProxyConfigServiceFixed(net::ProxyConfig::CreateDirect())));
+  scoped_ptr<net::URLRequestContext> context(builder.Build());
+  const net::HttpNetworkSession::Params* params =
+      context->GetNetworkSessionParams();
+  // Check Quic Connection options.
+  net::QuicTagVector quic_connection_options;
+  quic_connection_options.push_back(net::kTIME);
+  quic_connection_options.push_back(net::kTBBR);
+  quic_connection_options.push_back(net::kREJ);
+  EXPECT_EQ(quic_connection_options, params->quic_connection_options);
+
+  // Check store_server_configs_in_properties

[mef, 2015/11/18 20:33:25]

nit: End comment with period, also below.

[xunjieli, 2015/11/18 20:58:44]

Done.

+  EXPECT_TRUE(params->quic_store_server_configs_in_properties);
+
+  // Check delay_tcp_race
+  EXPECT_TRUE(params->quic_delay_tcp_race);
+}
+
+}  // namespace cronet