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

Unified Diff: chrome/browser/ui/website_settings/website_settings_unittest.cc

Issue 620983002: Add Permissions to the PageInfo dialog on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@page_info_dialog_shell_only_v2
Patch Set: Final rebase Created 6 years, 2 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/ui/website_settings/website_settings_unittest.cc
diff --git a/chrome/browser/ui/website_settings/website_settings_unittest.cc b/chrome/browser/ui/website_settings/website_settings_unittest.cc
index f979fcc66e78ff39a4ba678c6573bc97194fc6ff..7426a87f66b3d1acc3f01e5833f122631974a8e0 100644
--- a/chrome/browser/ui/website_settings/website_settings_unittest.cc
+++ b/chrome/browser/ui/website_settings/website_settings_unittest.cc
@@ -118,6 +118,8 @@ class WebsiteSettingsTest : public ChromeRenderViewHostTestHarness {
EXPECT_CALL(*mock_ui, SetFirstVisit(base::string16()));
}
+ void SetURL(std::string url) { url_ = GURL(url); }
+
const GURL& url() const { return url_; }
MockCertStore* cert_store() { return &cert_store_; }
int cert_id() { return cert_id_; }
@@ -391,3 +393,23 @@ TEST_F(WebsiteSettingsTest, ShowInfoBar) {
infobar_service()->RemoveInfoBar(infobar_service()->infobar_at(0));
}
+
+TEST_F(WebsiteSettingsTest, AboutBlankPage) {
+ SetURL("about:blank");
+ SetDefaultUIExpectations(mock_ui());
+ EXPECT_EQ(WebsiteSettings::SITE_CONNECTION_STATUS_INTERNAL_PAGE,
+ website_settings()->site_connection_status());
+ EXPECT_EQ(WebsiteSettings::SITE_IDENTITY_STATUS_INTERNAL_PAGE,
+ website_settings()->site_identity_status());
+ EXPECT_EQ(base::string16(), website_settings()->organization_name());
+}
+
+TEST_F(WebsiteSettingsTest, InternalPage) {
+ SetURL("chrome://bookmarks");
+ SetDefaultUIExpectations(mock_ui());
+ EXPECT_EQ(WebsiteSettings::SITE_CONNECTION_STATUS_INTERNAL_PAGE,
+ website_settings()->site_connection_status());
+ EXPECT_EQ(WebsiteSettings::SITE_IDENTITY_STATUS_INTERNAL_PAGE,
+ website_settings()->site_identity_status());
+ EXPECT_EQ(base::string16(), website_settings()->organization_name());
+}

Powered by Google App Engine
This is Rietveld 408576698