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

Unified Diff: chrome/browser/ssl/ssl_browser_tests.cc

Issue 1314843007: Refactor connection_security into SecurityStateModel (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: create SecurityStateModel for chromeos login webview 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: chrome/browser/ssl/ssl_browser_tests.cc
diff --git a/chrome/browser/ssl/ssl_browser_tests.cc b/chrome/browser/ssl/ssl_browser_tests.cc
index d817a77cb1b5215198c521518052fe4eec6de847..453eb77b46e89858555199a9b3bff8e12d013438 100644
--- a/chrome/browser/ssl/ssl_browser_tests.cc
+++ b/chrome/browser/ssl/ssl_browser_tests.cc
@@ -31,7 +31,7 @@
#include "chrome/browser/ssl/certificate_reporting_test_utils.h"
#include "chrome/browser/ssl/chrome_ssl_host_state_delegate.h"
#include "chrome/browser/ssl/common_name_mismatch_handler.h"
-#include "chrome/browser/ssl/connection_security.h"
+#include "chrome/browser/ssl/security_state_model.h"
#include "chrome/browser/ssl/ssl_blocking_page.h"
#include "chrome/browser/ssl/ssl_error_classification.h"
#include "chrome/browser/ssl/ssl_error_handler.h"
@@ -1007,27 +1007,26 @@ IN_PROC_BROWSER_TEST_F(SSLUITest, TestMarkNonSecureAs) {
base::FieldTrialList::CreateFieldTrial(
"MarkNonSecureAs", switches::kMarkNonSecureAsNonSecure);
+ content::WebContents* contents =
+ browser()->tab_strip_model()->GetActiveWebContents();
+ ASSERT_TRUE(contents);
+
+ SecurityStateModel* model = SecurityStateModel::FromWebContents(contents);
+ ASSERT_TRUE(model);
+
ui_test_utils::NavigateToURL(browser(), GURL("file:/"));
- EXPECT_EQ(connection_security::NONE,
- connection_security::GetSecurityLevelForWebContents(
- browser()->tab_strip_model()->GetActiveWebContents()));
+ EXPECT_EQ(SecurityStateModel::NONE, model->security_info().security_level);
ui_test_utils::NavigateToURL(browser(), GURL("about:blank"));
- EXPECT_EQ(connection_security::NONE,
- connection_security::GetSecurityLevelForWebContents(
- browser()->tab_strip_model()->GetActiveWebContents()));
+ EXPECT_EQ(SecurityStateModel::NONE, model->security_info().security_level);
ui_test_utils::NavigateToURL(browser(), GURL("data:text/plain,hello"));
- EXPECT_EQ(connection_security::NONE,
- connection_security::GetSecurityLevelForWebContents(
- browser()->tab_strip_model()->GetActiveWebContents()));
+ EXPECT_EQ(SecurityStateModel::NONE, model->security_info().security_level);
ui_test_utils::NavigateToURL(
browser(),
GURL("blob:chrome%3A//newtab/49a463bb-fac8-476c-97bf-5d7076c3ea1a"));
- EXPECT_EQ(connection_security::NONE,
- connection_security::GetSecurityLevelForWebContents(
- browser()->tab_strip_model()->GetActiveWebContents()));
+ EXPECT_EQ(SecurityStateModel::NONE, model->security_info().security_level);
}
#if defined(USE_NSS_CERTS)
« no previous file with comments | « chrome/browser/ssl/security_state_model_browser_tests.cc ('k') | chrome/browser/ui/android/bluetooth_chooser_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698