website_settings: Fix Website Settings popup for chrome://bookmarks.

chrome/browser/ui/cocoa/website_settings_bubble_controller.mm

Index: chrome/browser/ui/cocoa/website_settings_bubble_controller.mm
diff --git a/chrome/browser/ui/cocoa/website_settings_bubble_controller.mm b/chrome/browser/ui/cocoa/website_settings_bubble_controller.mm
index a0453f52abd620ba8f39121a6b93b9c454074ea2..fd3cb0725d74dfbcb0060c220b2d7a850b13d83d 100644
--- a/chrome/browser/ui/cocoa/website_settings_bubble_controller.mm
+++ b/chrome/browser/ui/cocoa/website_settings_bubble_controller.mm
@@ -1138,7 +1138,8 @@ void WebsiteSettingsUIBridge::Show(gfx::NativeWindow parent,
                                    const GURL& url,
                                    const content::SSLStatus& ssl) {
   bool is_internal_page = url.SchemeIs(chrome::kChromeInternalScheme) ||

[tfarina, 2012/09/24 19:19:03]

Why InternalChromePage is not used here?

[markusheintz_, 2012/09/24 19:21:02]

InternalChromePage should be used here as well. Please use it. Thanks!

-                          url.SchemeIs(chrome::kChromeUIScheme);
+                          url.SchemeIs(chrome::kChromeUIScheme) ||
+                          url.SchemeIs(chrome::kExtensionScheme);
 
   // Create the bridge. This will be owned by the bubble controller.
   WebsiteSettingsUIBridge* bridge = new WebsiteSettingsUIBridge();