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

Unified Diff: chrome/browser/ui/page_info/page_info.cc

Issue 2944973002: Remove IS_IOS checks in chrome/ (Closed)
Patch Set: Created 3 years, 6 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 | « chrome/browser/ui/browser_ui_prefs.cc ('k') | chrome/browser/ui/page_info/page_info_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/page_info/page_info.cc
diff --git a/chrome/browser/ui/page_info/page_info.cc b/chrome/browser/ui/page_info/page_info.cc
index a0e2e5d4452c6e40543270d048f1529024604ba8..17e400155746b9ff04e4bbd733d569ac39cca433 100644
--- a/chrome/browser/ui/page_info/page_info.cc
+++ b/chrome/browser/ui/page_info/page_info.cc
@@ -440,7 +440,7 @@ void PageInfo::OpenSiteSettingsView() {
void PageInfo::Init(const GURL& url,
const security_state::SecurityInfo& security_info) {
-#if !defined(OS_ANDROID) && !defined(OS_IOS)
+#if !defined(OS_ANDROID)
// On desktop, internal URLs aren't handled by this class. Instead, a
// custom and simpler bubble is shown.
DCHECK(!url.SchemeIs(content::kChromeUIScheme) &&
@@ -449,9 +449,9 @@ void PageInfo::Init(const GURL& url,
!url.SchemeIs(content_settings::kExtensionScheme));
#endif
- bool isChromeUINativeScheme = false;
+ bool is_chrome_ui_native_scheme = false;
#if defined(OS_ANDROID)
- isChromeUINativeScheme = url.SchemeIs(chrome::kChromeUINativeScheme);
+ is_chrome_ui_native_scheme = url.SchemeIs(chrome::kChromeUINativeScheme);
#endif
security_level_ = security_info.security_level;
@@ -469,7 +469,7 @@ void PageInfo::Init(const GURL& url,
return;
}
- if (url.SchemeIs(content::kChromeUIScheme) || isChromeUINativeScheme) {
+ if (url.SchemeIs(content::kChromeUIScheme) || is_chrome_ui_native_scheme) {
site_identity_status_ = SITE_IDENTITY_STATUS_INTERNAL_PAGE;
site_identity_details_ =
l10n_util::GetStringUTF16(IDS_PAGE_INFO_INTERNAL_PAGE);
« no previous file with comments | « chrome/browser/ui/browser_ui_prefs.cc ('k') | chrome/browser/ui/page_info/page_info_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698