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

Unified Diff: content/common/ssl_status_serialization_unittest.cc

Issue 1957483003: Removal of SignedCertificateTimestampStore (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixed IPC problem Created 4 years, 7 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
« no previous file with comments | « content/common/ssl_status_serialization.cc ('k') | content/content_browser.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/ssl_status_serialization_unittest.cc
diff --git a/content/common/ssl_status_serialization_unittest.cc b/content/common/ssl_status_serialization_unittest.cc
index d479518add63ab52a74e694f2eff5b17b7434d8f..d230c329e3481f276a1f50ae2c3f7746a38a74a1 100644
--- a/content/common/ssl_status_serialization_unittest.cc
+++ b/content/common/ssl_status_serialization_unittest.cc
@@ -18,8 +18,9 @@ void SetTestStatus(SSLStatus* status) {
status->security_bits = 80;
status->key_exchange_info = 23;
status->connection_status = net::SSL_CONNECTION_VERSION_TLS1_2;
- status->signed_certificate_timestamp_ids.push_back(
- SignedCertificateTimestampIDAndStatus(1, net::ct::SCT_STATUS_OK));
+ status->num_unknown_scts = 0;
+ status->num_invalid_scts = 0;
+ status->num_valid_scts = 1;
}
bool SSLStatusAreEqual(const SSLStatus& a, const SSLStatus &b) {
@@ -30,13 +31,15 @@ bool SSLStatusAreEqual(const SSLStatus& a, const SSLStatus &b) {
std::ostream& operator<<(std::ostream& os, const SSLStatus& status) {
return os << "Security Style: " << status.security_style
- << "\nCert ID: " << status.cert_id
- << "\nCert Status: " << status.cert_status
- << "\nSecurity bits: " << status.security_bits
- << "\nKey exchange info: " << status.key_exchange_info
- << "\nConnection status: " << status.connection_status
- << "\nContent Status: " << status.content_status
- << "\nNumber of SCTs: " << status.signed_certificate_timestamp_ids.size();
+ << "\nCert ID: " << status.cert_id
+ << "\nCert Status: " << status.cert_status
+ << "\nSecurity bits: " << status.security_bits
+ << "\nKey exchange info: " << status.key_exchange_info
+ << "\nConnection status: " << status.connection_status
+ << "\nContent Status: " << status.content_status
+ << "\nNumber of unknown SCTs: " << status.num_unknown_scts
+ << "\nNumber of invalid SCTs: " << status.num_invalid_scts
+ << "\nNumber of valid SCTs: " << status.num_valid_scts;
}
// Test that a valid serialized SSLStatus returns true on
@@ -50,8 +53,6 @@ TEST(SSLStatusSerializationTest, DeserializeSerializedStatus) {
SSLStatus deserialized;
ASSERT_TRUE(DeserializeSecurityInfo(serialized, &deserialized));
EXPECT_PRED2(SSLStatusAreEqual, status, deserialized);
- EXPECT_EQ(SignedCertificateTimestampIDAndStatus(1, net::ct::SCT_STATUS_OK),
- deserialized.signed_certificate_timestamp_ids[0]);
// Test that |content_status| has the default (initialized) value.
EXPECT_EQ(SSLStatus::NORMAL_CONTENT, deserialized.content_status);
}
« no previous file with comments | « content/common/ssl_status_serialization.cc ('k') | content/content_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698