Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1492)

Unified Diff: content/public/common/ssl_status.h

Issue 1332633002: Add constructor to create SSLStatus from SSLInfo (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: temp: debug logging for mac/win test failure Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/common/ssl_status.h
diff --git a/content/public/common/ssl_status.h b/content/public/common/ssl_status.h
index f0a9b1fce02a2fa9f7c4908ccd195d466a313fda..153a74d92faccc59de793a88b36e736b7ac272c3 100644
--- a/content/public/common/ssl_status.h
+++ b/content/public/common/ssl_status.h
@@ -10,6 +10,10 @@
#include "content/public/common/signed_certificate_timestamp_id_and_status.h"
#include "net/cert/cert_status_flags.h"
+namespace net {
+class SSLInfo;
+}
+
namespace content {
// Collects the SSL information for this NavigationEntry.
@@ -30,6 +34,11 @@ struct CONTENT_EXPORT SSLStatus {
};
SSLStatus();
+ SSLStatus(
+ SecurityStyle security_style,
+ int cert_id,
+ SignedCertificateTimestampIDStatusList signed_certificate_timestamp_ids,
+ const net::SSLInfo& ssl_info);
~SSLStatus();
bool Equals(const SSLStatus& status) const {
meacer 2015/09/09 18:26:08 Off topic: Odd, I'm wondering why this isn't an op
estark 2015/09/10 14:32:05 Yeah I wondered that too... no idea.
@@ -37,6 +46,7 @@ struct CONTENT_EXPORT SSLStatus {
cert_id == status.cert_id &&
cert_status == status.cert_status &&
security_bits == status.security_bits &&
+ connection_status == status.connection_status &&
content_status == status.content_status &&
signed_certificate_timestamp_ids ==
status.signed_certificate_timestamp_ids;

Powered by Google App Engine
This is Rietveld 408576698