Fix nits related to SSLHostInfo.

net/socket/ssl_host_info.cc

 // Copyright (c) 2011 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 "net/socket/ssl_host_info.h"
 
 #include "base/metrics/histogram.h"
 #include "base/pickle.h"
 #include "base/string_piece.h"
 #include "net/base/dns_util.h"
@@ skipping 76 matching lines @@
     return false;
   }
 
   for (int i = 0; i < num_der_certs; i++) {
     std::string der_cert;
     if (!p.ReadString(&iter, &der_cert))
       return false;
     state->certs.push_back(der_cert);
   }
 
+  // Ignore obsolete members of the State structure.
   std::string throwaway_string;
   bool throwaway_bool;
+  // This was state->server_hello.
   if (!p.ReadString(&iter, &throwaway_string))
     return false;
 
+  // This was state->npn_valid.
   if (!p.ReadBool(&iter, &throwaway_bool))
     return false;
 
   if (throwaway_bool) {
     int throwaway_int;
+    // These were state->npn_status and state->npn_protocol.
     if (!p.ReadInt(&iter, &throwaway_int) ||
         !p.ReadString(&iter, &throwaway_string)) {
       return false;
     }
   }
 
   if (!state->certs.empty()) {
     std::vector<base::StringPiece> der_certs(state->certs.size());
     for (size_t i = 0; i < state->certs.size(); i++)
       der_certs[i] = state->certs[i];
@@ skipping 37 matching lines @@
 
   if (!p.WriteInt(state_.certs.size()))
     return "";
 
   for (std::vector<std::string>::const_iterator
        i = state_.certs.begin(); i != state_.certs.end(); i++) {
     if (!p.WriteString(*i))
       return "";
   }
 
+  // Write dummy values for obsolete members of the State structure:
+  // state->server_hello and state->npn_valid.
   if (!p.WriteString("") ||
       !p.WriteBool(false)) {
     return "";
   }
 
   return std::string(reinterpret_cast<const char *>(p.data()), p.size());
 }
 
 const CertVerifyResult& SSLHostInfo::cert_verify_result() const {
   return cert_verify_result_;
@@ skipping 27 matching lines @@
   if (cert_verification_callback_) {
     CompletionCallback* callback = cert_verification_callback_;
     cert_verification_callback_ = NULL;
     callback->Run(rv);
   }
 }
 
 SSLHostInfoFactory::~SSLHostInfoFactory() {}
 
 }  // namespace net