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

Unified Diff: components/security_state/core/security_state_unittest.cc

Issue 2635423002: Enable HTTP-bad Phase 1 by default except iOS (Closed)
Patch Set: fixes Created 3 years, 11 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 | « components/security_state/core/security_state.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/security_state/core/security_state_unittest.cc
diff --git a/components/security_state/core/security_state_unittest.cc b/components/security_state/core/security_state_unittest.cc
index f6a899cf4491611c51cfea36038bff00111226ea..13409f3357f1e6756193495f14da0fb1f4593e48 100644
--- a/components/security_state/core/security_state_unittest.cc
+++ b/components/security_state/core/security_state_unittest.cc
@@ -274,24 +274,6 @@ TEST(SecurityStateTest, CreditCardFieldWarning) {
EXPECT_EQ(HTTP_SHOW_WARNING, security_info.security_level);
}
-// Tests that neither password nor credit fields cause the security
-// level to be downgraded to HTTP_SHOW_WARNING when the command-line switch
-// is NOT set.
-TEST(SecurityStateTest, HttpWarningNotSetWithoutSwitch) {
- TestSecurityStateHelper helper;
- helper.UseHttpUrl();
- helper.set_displayed_password_field_on_http(true);
- SecurityInfo security_info;
- helper.GetSecurityInfo(&security_info);
- EXPECT_TRUE(security_info.displayed_password_field_on_http);
- EXPECT_EQ(NONE, security_info.security_level);
-
- helper.set_displayed_credit_card_field_on_http(true);
- helper.GetSecurityInfo(&security_info);
- EXPECT_TRUE(security_info.displayed_credit_card_field_on_http);
- EXPECT_EQ(NONE, security_info.security_level);
-}
-
// Tests that neither |displayed_password_field_on_http| nor
// |displayed_credit_card_field_on_http| is set when the corresponding
// VisibleSecurityState flags are not set.
« no previous file with comments | « components/security_state/core/security_state.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698