Make ParseAndSetExperimentalOptions() a member of URLRequestContextConfig

components/cronet/url_request_context_config.h

 // 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.
 
 #ifndef COMPONENTS_CRONET_URL_REQUEST_CONTEXT_CONFIG_H_
 #define COMPONENTS_CRONET_URL_REQUEST_CONTEXT_CONFIG_H_
 
 #include <memory>
 #include <string>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_vector.h"
 #include "base/time/time.h"
+#include "base/values.h"
 #include "net/base/hash_value.h"
 #include "net/cert/cert_verifier.h"
 
 namespace base {
-class DictionaryValue;
 class SequencedTaskRunner;
 }  // namespace base
 
 namespace net {
 class CertVerifier;
 class NetLog;
 class URLRequestContextBuilder;
 }  // namespace net
 
 namespace cronet {
 
 // Common configuration parameters used by Cronet to configure
 // URLRequestContext.
+// TODO(mgersh): This shouldn't be a struct, and experimental option parsing
+// should be kept more separate from applying the configuration.
 struct URLRequestContextConfig {
   // Type of HTTP cache.
   // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.net.impl
   enum HttpCacheType {
     // No HTTP cache.
     DISABLED,
     // HTTP cache persisted to disk.
     DISK,
     // HTTP cache kept in memory.
     MEMORY,
@@ skipping 116 matching lines @@
   // Data to populte CertVerifierCache.
   const std::string cert_verifier_data;
 
   // App-provided list of servers that support QUIC.
   ScopedVector<QuicHint> quic_hints;
 
   // The list of public key pins.
   ScopedVector<Pkp> pkp_list;
 
   // Experimental options that are recognized by the config parser.
-  std::unique_ptr<base::DictionaryValue> effective_experimental_options;
+  std::unique_ptr<base::DictionaryValue> effective_experimental_options =
+      nullptr;
 
  private:
+  // Parses experimental options and makes appropriate changes to settings in
+  // the URLRequestContextConfig and URLRequestContextBuilder.
+  void ParseAndSetExperimentalOptions(
+      net::URLRequestContextBuilder* context_builder,
+      net::NetLog* net_log,
+      const scoped_refptr<base::SequencedTaskRunner>& file_task_runner);
+
   DISALLOW_COPY_AND_ASSIGN(URLRequestContextConfig);
 };
 
 // Stores intermediate state for URLRequestContextConfig.  Initializes with
 // (mostly) sane defaults, then the appropriate member variables can be
 // modified, and it can be finalized with Build().
 struct URLRequestContextConfigBuilder {
   URLRequestContextConfigBuilder();
   ~URLRequestContextConfigBuilder();
 
@@ skipping 44 matching lines @@
   // Data to populate CertVerifierCache.
   std::string cert_verifier_data = "";
 
  private:
   DISALLOW_COPY_AND_ASSIGN(URLRequestContextConfigBuilder);
 };
 
 }  // namespace cronet
 
 #endif  // COMPONENTS_CRONET_URL_REQUEST_CONTEXT_CONFIG_H_