Clang thinks my change made SSLSocketDataProvider complicated.

BUG=clang
TEST=n/a


Review URL: http://codereview.chromium.org/6639017

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77427 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 12 insertions, 5 deletions

diff --git a/net/socket/socket_test_util.cc b/net/socket/socket_test_util.cc
index 2ed4b83..fb4c7ae 100644
--- a/net/socket/socket_test_util.cc
+++ b/net/socket/socket_test_util.cc
@@ -235,6 +235,16 @@ void DynamicSocketDataProvider::SimulateRead(const char* data,
   reads_.push_back(MockRead(true, data, length));
 }
 
+SSLSocketDataProvider::SSLSocketDataProvider(bool async, int result)
+    : connect(async, result),
+      next_proto_status(SSLClientSocket::kNextProtoUnsupported),
+      was_npn_negotiated(false),
+      cert_request_info(NULL) {
+}
+
+SSLSocketDataProvider::~SSLSocketDataProvider() {
+}
+
 DelayedSocketData::DelayedSocketData(
     int write_delay, MockRead* reads, size_t reads_count,
     MockWrite* writes, size_t writes_count)
diff --git a/net/socket/socket_test_util.h b/net/socket/socket_test_util.h
index 7e00029..c661ebb 100644
--- a/net/socket/socket_test_util.h
+++ b/net/socket/socket_test_util.h
@@ -243,11 +243,8 @@ class DynamicSocketDataProvider : public SocketDataProvider {
 // SSLSocketDataProviders only need to keep track of the return code from calls
 // to Connect().
 struct SSLSocketDataProvider {
-  SSLSocketDataProvider(bool async, int result)
-      : connect(async, result),
-        next_proto_status(SSLClientSocket::kNextProtoUnsupported),
-        was_npn_negotiated(false),
-        cert_request_info(NULL) { }
+  SSLSocketDataProvider(bool async, int result);
+  ~SSLSocketDataProvider();
 
   MockConnect connect;
   SSLClientSocket::NextProtoStatus next_proto_status;