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

Side by Side Diff: chrome/browser/ui/website_settings/website_settings_unittest.cc

Issue 2397273002: Show a custom page info bubble for chrome-devtools:// URLs (Closed)
Patch Set: Rebase Created 4 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/ui/website_settings/website_settings.cc ('k') | chrome/test/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/website_settings/website_settings.h" 5 #include "chrome/browser/ui/website_settings/website_settings.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/at_exit.h" 10 #include "base/at_exit.h"
(...skipping 848 matching lines...) Expand 10 before | Expand all | Expand 10 after
859 TEST_F(WebsiteSettingsTest, AboutBlankPage) { 859 TEST_F(WebsiteSettingsTest, AboutBlankPage) {
860 SetURL("about:blank"); 860 SetURL("about:blank");
861 SetDefaultUIExpectations(mock_ui()); 861 SetDefaultUIExpectations(mock_ui());
862 EXPECT_EQ(WebsiteSettings::SITE_CONNECTION_STATUS_UNENCRYPTED, 862 EXPECT_EQ(WebsiteSettings::SITE_CONNECTION_STATUS_UNENCRYPTED,
863 website_settings()->site_connection_status()); 863 website_settings()->site_connection_status());
864 EXPECT_EQ(WebsiteSettings::SITE_IDENTITY_STATUS_NO_CERT, 864 EXPECT_EQ(WebsiteSettings::SITE_IDENTITY_STATUS_NO_CERT,
865 website_settings()->site_identity_status()); 865 website_settings()->site_identity_status());
866 EXPECT_EQ(base::string16(), website_settings()->organization_name()); 866 EXPECT_EQ(base::string16(), website_settings()->organization_name());
867 } 867 }
868 868
869 // On desktop, internal URLs aren't handled by WebsiteSettings class. Instead, a
870 // custom and simpler popup is shown, so no need to test.
871 #if defined(OS_ANDROID) || defined(OS_IOS)
869 TEST_F(WebsiteSettingsTest, InternalPage) { 872 TEST_F(WebsiteSettingsTest, InternalPage) {
870 SetURL("chrome://bookmarks"); 873 SetURL("chrome://bookmarks");
871 SetDefaultUIExpectations(mock_ui()); 874 SetDefaultUIExpectations(mock_ui());
872 EXPECT_EQ(WebsiteSettings::SITE_CONNECTION_STATUS_INTERNAL_PAGE, 875 EXPECT_EQ(WebsiteSettings::SITE_CONNECTION_STATUS_INTERNAL_PAGE,
873 website_settings()->site_connection_status()); 876 website_settings()->site_connection_status());
874 EXPECT_EQ(WebsiteSettings::SITE_IDENTITY_STATUS_INTERNAL_PAGE, 877 EXPECT_EQ(WebsiteSettings::SITE_IDENTITY_STATUS_INTERNAL_PAGE,
875 website_settings()->site_identity_status()); 878 website_settings()->site_identity_status());
876 EXPECT_EQ(base::string16(), website_settings()->organization_name()); 879 EXPECT_EQ(base::string16(), website_settings()->organization_name());
877 } 880 }
881 #endif
OLDNEW
« no previous file with comments | « chrome/browser/ui/website_settings/website_settings.cc ('k') | chrome/test/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698