Handle about:blank in the Origin Info Bubble better.

chrome/browser/ui/website_settings/website_settings.cc

Index: chrome/browser/ui/website_settings/website_settings.cc
diff --git a/chrome/browser/ui/website_settings/website_settings.cc b/chrome/browser/ui/website_settings/website_settings.cc
index 9996abb1bd02d3c7a724eed37bce73aed2921872..89130297005b814d37327175333975c0fafc5b2b 100644
--- a/chrome/browser/ui/website_settings/website_settings.cc
+++ b/chrome/browser/ui/website_settings/website_settings.cc
@@ -364,17 +364,10 @@ void WebsiteSettings::Init(
   if (url.SchemeIs(url::kAboutScheme)) {
     // All about: URLs except about:blank are redirected.
     DCHECK_EQ(url::kAboutBlankURL, url.spec());
-    site_identity_status_ = SITE_IDENTITY_STATUS_NO_CERT;
-    site_identity_details_ =
-        l10n_util::GetStringUTF16(IDS_PAGE_INFO_SECURITY_TAB_INSECURE_IDENTITY);
-    site_connection_status_ = SITE_CONNECTION_STATUS_UNENCRYPTED;
-    site_connection_details_ = l10n_util::GetStringFUTF16(
-        IDS_PAGE_INFO_SECURITY_TAB_NOT_ENCRYPTED_CONNECTION_TEXT,
-        UTF8ToUTF16(url.spec()));
-    return;
   }

[meacer, 2016/03/17 22:26:24]

This conflicts with
https://chromium.googlesource.com/chromium/src/+/60886a6e1591b314f9702fc5adc8...,

we explicitly wanted not to show "This is an internal chrome page" for
about:blank URLs. Note that about:blank URLs are actually controlled by their
opener:

w = window.open("about:blank")
w.document.write("hello")

will modify the contents of the about:blank page.

 
-  if (url.SchemeIs(content::kChromeUIScheme) || isChromeUINativeScheme) {
+  if (url.SchemeIs(content::kChromeUIScheme) ||
+      url.SchemeIs(url::kAboutScheme) || isChromeUINativeScheme) {
     site_identity_status_ = SITE_IDENTITY_STATUS_INTERNAL_PAGE;
     site_identity_details_ =
         l10n_util::GetStringUTF16(IDS_PAGE_INFO_INTERNAL_PAGE);