Remove JS dialog dependency from content.

chrome/browser/external_tab_container_win.h

Index: chrome/browser/external_tab_container_win.h
diff --git a/chrome/browser/external_tab_container_win.h b/chrome/browser/external_tab_container_win.h
index e5435771f1c57d1cacee50f42baa1d877a9eab19..ba0fd6e86a8237afa1017802f20f459f9175166a 100644
--- a/chrome/browser/external_tab_container_win.h
+++ b/chrome/browser/external_tab_container_win.h
@@ -170,6 +170,9 @@ class ExternalTabContainer : public TabContentsDelegate,
   void RegisterRenderViewHost(RenderViewHost* render_view_host);
   void UnregisterRenderViewHost(RenderViewHost* render_view_host);
 
+  virtual content::JavaScriptDialogCreator* GetJavaScriptDialogCreator(

[jam, 2011/06/03 00:21:45]

this should be with the other TabContentsDelegate methods right?  (i.e. at 169,
and looks like ShowRepostFormWarningDialog is missing a virtual)

+      ) OVERRIDE;

[jam, 2011/06/03 00:21:45]

nit: this seems awkward to read.  do you think it'd be more readable if the
function name was on the following line?

+
   // Overridden from TabContentsObserver:
   // IPC::Channel::Listener implementation.
   virtual bool OnMessageReceived(const IPC::Message& message);