Expose SSLInfo::pkp_bypassed to devtools

components/security_state/security_state_model.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/security_state/security_state_model.h"
 
 #include <stdint.h>
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
@@ skipping 166 matching lines @@
   security_info->cert_id = visible_security_state.cert_id;
   security_info->sha1_deprecation_status =
       GetSHA1DeprecationStatus(cert, visible_security_state);
   security_info->mixed_content_status =
       GetMixedContentStatus(visible_security_state);
   security_info->security_bits = visible_security_state.security_bits;
   security_info->connection_status = visible_security_state.connection_status;
   security_info->cert_status = visible_security_state.cert_status;
   security_info->scheme_is_cryptographic =
       visible_security_state.url.SchemeIsCryptographic();
+  security_info->pkp_bypassed = visible_security_state.pkp_bypassed;
   security_info->is_secure_protocol_and_ciphersuite =
       (net::SSLConnectionStatusToVersion(security_info->connection_status) >=
            net::SSL_CONNECTION_VERSION_TLS1_2 &&
        net::IsSecureTLSCipherSuite(net::SSLConnectionStatusToCipherSuite(
            security_info->connection_status)));
 
   security_info->sct_verify_statuses =
       visible_security_state.sct_verify_statuses;
 
   security_info->security_level =
@@ skipping 13 matching lines @@
 
 SecurityStateModel::SecurityInfo::SecurityInfo()
     : security_level(SecurityStateModel::NONE),
       sha1_deprecation_status(SecurityStateModel::NO_DEPRECATED_SHA1),
       mixed_content_status(SecurityStateModel::NO_MIXED_CONTENT),
       scheme_is_cryptographic(false),
       cert_status(0),
       cert_id(0),
       security_bits(-1),
       connection_status(0),
-      is_secure_protocol_and_ciphersuite(false) {}
+      is_secure_protocol_and_ciphersuite(false),
+      pkp_bypassed(false) {}
 
 SecurityStateModel::SecurityInfo::~SecurityInfo() {}
 
 SecurityStateModel::SecurityStateModel() {}
 
 SecurityStateModel::~SecurityStateModel() {}
 
 const SecurityStateModel::SecurityInfo& SecurityStateModel::GetSecurityInfo()
     const {
   scoped_refptr<net::X509Certificate> cert = nullptr;
@@ skipping 25 matching lines @@
 }
 
 SecurityStateModel::VisibleSecurityState::VisibleSecurityState()
     : initialized(false),
       initial_security_level(SecurityStateModel::NONE),
       cert_id(0),
       cert_status(0),
       connection_status(0),
       security_bits(-1),
       displayed_mixed_content(false),
       ran_mixed_content(false) {}

[estark, 2016/06/28 02:43:52]

initialize |pkp_bypassed| here, and also check it in operator== below

[dadrian, 2016/06/28 18:45:58]

Done.

 
 SecurityStateModel::VisibleSecurityState::~VisibleSecurityState() {}
 
 bool SecurityStateModel::VisibleSecurityState::operator==(
     const SecurityStateModel::VisibleSecurityState& other) const {
   return (url == other.url &&
           initial_security_level == other.initial_security_level &&
           cert_id == other.cert_id && cert_status == other.cert_status &&
           connection_status == other.connection_status &&
           security_bits == other.security_bits &&
           sct_verify_statuses == other.sct_verify_statuses &&
           displayed_mixed_content == other.displayed_mixed_content &&
           ran_mixed_content == other.ran_mixed_content);
 }
 
 }  // namespace security_state