Show a custom page info bubble for chrome-devtools:// URLs

chrome/browser/ui/website_settings/website_settings_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/website_settings/website_settings.h"
 
 #include <string>
 #include <vector>
 
 #include "base/at_exit.h"
@@ skipping 847 matching lines @@
 
 TEST_F(WebsiteSettingsTest, AboutBlankPage) {
   SetURL("about:blank");
   SetDefaultUIExpectations(mock_ui());
   EXPECT_EQ(WebsiteSettings::SITE_CONNECTION_STATUS_UNENCRYPTED,
             website_settings()->site_connection_status());
   EXPECT_EQ(WebsiteSettings::SITE_IDENTITY_STATUS_NO_CERT,
             website_settings()->site_identity_status());
   EXPECT_EQ(base::string16(), website_settings()->organization_name());
 }
-
-TEST_F(WebsiteSettingsTest, InternalPage) {
-  SetURL("chrome://bookmarks");

[elawrence, 2016/10/07 21:04:01]

Was this test removed because it's obsoleted by the chrome://settings case
inside website_settings_popup_view_interactive_uitest.cc? 

[meacer, 2016/10/10 18:09:19]

I removed it because I thought it would never be possible for
website_settings.cc to display a chrome:// URL. That's not true on Android, so
reverting this change too.

-  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());
-}