Remove any platform specific code from the chrome/browser/website_settings.* .

chrome/browser/ui/gtk/browser_window_gtk.cc

Index: chrome/browser/ui/gtk/browser_window_gtk.cc
diff --git a/chrome/browser/ui/gtk/browser_window_gtk.cc b/chrome/browser/ui/gtk/browser_window_gtk.cc
index 1bc976a5d3357c0070810067f9965590af0ca921..18c965386caf27c766b2750de7a7e24e39d3af4a 100644
--- a/chrome/browser/ui/gtk/browser_window_gtk.cc
+++ b/chrome/browser/ui/gtk/browser_window_gtk.cc
@@ -70,6 +70,7 @@
 #include "chrome/browser/ui/gtk/tabs/tab_strip_gtk.h"
 #include "chrome/browser/ui/gtk/task_manager_gtk.h"
 #include "chrome/browser/ui/gtk/update_recommended_dialog.h"
+#include "chrome/browser/ui/gtk/website_settings_popup_gtk.h"
 #include "chrome/browser/ui/omnibox/location_bar.h"
 #include "chrome/browser/ui/omnibox/omnibox_view.h"
 #include "chrome/browser/ui/page_info_bubble.h"
@@ -77,7 +78,6 @@
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/webui/task_manager/task_manager_dialog.h"
 #include "chrome/browser/web_applications/web_app.h"
-#include "chrome/browser/website_settings.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
@@ -1139,11 +1139,11 @@ void BrowserWindowGtk::ShowWebsiteSettings(
     const GURL& url,
     const content::SSLStatus& ssl,
     bool show_history) {
-    WebsiteSettings::Show(GetNativeHandle(),
-                          profile,
-                          tab_contents_wrapper,
-                          url,
-                          ssl);
+    WebsiteSettingsPopupGtk::Show(GetNativeHandle(),

[tfarina, 2012/05/31 17:34:19]

This works for me. But it isn't what I suggested.

What I have in mind is:

WebsiteSettings::Show(...);

and in website_settings_popup_gtk.cc (at the end of the file):

void WebsiteSettings::Show(...) {
  new WebsiteSettingsPopupGtk(...);
}

and in chrome/browser/website_settings.h

#if defined(TOOLKIT_GTK)
static void Show(...); // the gtk declaration, which unfortunately differ from
views declaration because BubbleDelegate requires an anchor_view not a
gfx::NativeWindow :/
#elif defined(TOOLKIT_VIEWS)
static void Show(...);
#endif

That way you can open this popup from other places outside of
chrome/browser/ui/gtk.

If can take a look at how speech and intents do this in chrome/browser/speech,
and chrome/browser/ui/intents/ and various browser::ShowFooDialog() calls.

+                                  profile,
+                                  tab_contents_wrapper,
+                                  url,
+                                  ssl);
 }
 
 void BrowserWindowGtk::ShowAppMenu() {