Show extension uninstall dialog in browser window.

chrome/browser/extensions/chrome_extension_function_details.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_FUNCTION_DETAILS_H_
 #define CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_FUNCTION_DETAILS_H_
 
 #include "base/macros.h"
 
 class Browser;
@@ skipping 44 matching lines @@
   // TODO(stevenjb): Replace this with GetExtensionWindowController().
   Browser* GetCurrentBrowser() const;
 
   // Same as above but uses WindowControllerList instead of BrowserList.
   extensions::WindowController* GetExtensionWindowController() const;
 
   // Gets the "current" web contents if any. If there is no associated web
   // contents then defaults to the foremost one.
   content::WebContents* GetAssociatedWebContents();
 
+  // Gets the web contents if it's not from a background page. Otherwise this

[not at google - send to devlin, 2015/07/10 17:02:25]

Actually can we call this method something with a weaker guarantee than
"GetVisibleWebContents"? It's not really clear what a "visible web contents"
means to this function in particular. "Sender" is obvious. "Associated" is a
really bad name which is why we want to get rid of that function.

In fact, I'd even go so far as to make this return a NativeWindow, like
"GetNativeWindowForUI", and document it as finding a UI surface to display any
UI (like a permission prompt) for the extension calling this function. You could
also move the active browser check into here.

[wjywbs, 2015/07/10 19:54:22]

I renamed this to be "GetOriginWebContents" because media gallery only needs
WebContents but not NativeWindow. I added the "GetNativeWindowForUI" function.

+  // method will try to find the app's web contents by the extension id, or
+  // find the web contents from source_tab_id if it's not
+  // TabStripModel::kNoTab. If the web contents still can't be found, NULL will
+  // be returned.
+  content::WebContents* GetVisibleWebContents();
+
   // Returns a pointer to the associated UIThreadExtensionFunction
   UIThreadExtensionFunction* function() { return function_; }
   const UIThreadExtensionFunction* function() const { return function_; }
 
  private:
   // The function for which these details have been created. Must outlive the
   // ChromeExtensionFunctionDetails instance.
   UIThreadExtensionFunction* function_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeExtensionFunctionDetails);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_FUNCTION_DETAILS_H_