Pull shell window stuff out of ExtensionHost and put in ShellWindow

chrome/browser/extensions/extension_function_dispatcher.h

Index: chrome/browser/extensions/extension_function_dispatcher.h
diff --git a/chrome/browser/extensions/extension_function_dispatcher.h b/chrome/browser/extensions/extension_function_dispatcher.h
index 982910706267bc2389feed9f9f2b4ceb18f8d2a8..37faf1c1f4af3a684c60de2e0a664a5b037c1962 100644
--- a/chrome/browser/extensions/extension_function_dispatcher.h
+++ b/chrome/browser/extensions/extension_function_dispatcher.h
@@ -55,13 +55,13 @@ class ExtensionFunctionDispatcher
    public:
     // Returns the browser that this delegate is associated with, if any.
     // Returns NULL otherwise.
-    virtual Browser* GetBrowser() = 0;
+    virtual Browser* GetBrowser();

[jeremya, 2012/05/07 00:54:38]

This is going to conflict with stevenjb's patch:
http://codereview.chromium.org/10021071/

[benwells, 2012/05/07 08:46:15]

;_;

 
     // Asks the delegate for any relevant WebContents associated with this
     // context. For example, the WebbContents in which an infobar or
     // chrome-extension://<id> URL are being shown. Callers must check for a
     // NULL return value (as in the case of a background page).
-    virtual content::WebContents* GetAssociatedWebContents() const = 0;
+    virtual content::WebContents* GetAssociatedWebContents() const;
 
    protected:
     virtual ~Delegate() {}