Perform ClientHello padding if the field trial is enabled

net/ssl/ssl_socket_config_service.h

+// Copyright (c) 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.
+
+#ifndef NET_SSL_SSL_SOCKET_CONFIG_SERVICE_H_
+#define NET_SSL_SSL_SOCKET_CONFIG_SERVICE_H_
+
+#include "base/macros.h"
+#include "net/base/net_export.h"
+
+namespace net {
+
+class HostPortPair;
+
+class NET_EXPORT SSLSocketConfigService {
+ public:
+  SSLSocketConfigService() : enabled_(false) {}
+  virtual ~SSLSocketConfigService() {}
+
+  // Enables fast radio padding.
+  void EnableFastRadioPadding();
+  // Disables fast radio padding.
+  void DisableFastRadioPadding();

[Ryan Sleevi, 2015/01/30 22:11:32]

Why isn't this a single function with a bool arg?

+
+  // Checks to see whether the socket should be configured to use ClientHello
+  // padding.
+  bool UseFastRadioPadding(const HostPortPair& host_and_port);
+
+  // Returns whether this is a Google host/port or not. Matches host/port
+  // logic in ssl_client_socket_pool.cc

[Ryan Sleevi, 2015/01/30 22:11:32]

This comment seems very much a layering violation. Talk about what it does, not
about what something else unrelated does.

+  virtual bool IsGoogle(const HostPortPair& host_and_port);

[Ryan Sleevi, 2015/01/30 22:11:32]

Why is this virtual? "For Testing"?

+
+ private:
+  bool enabled_;
+
+  DISALLOW_COPY_AND_ASSIGN(SSLSocketConfigService);
+};
+
+}  // namespace net
+
+#endif  // NET_SSL_SSL_SOCKET_CONFIG_SERVICE_H_