Show extension uninstall dialog in browser window.

chrome/browser/extensions/chrome_extension_function_details.h

Index: chrome/browser/extensions/chrome_extension_function_details.h
diff --git a/chrome/browser/extensions/chrome_extension_function_details.h b/chrome/browser/extensions/chrome_extension_function_details.h
index 2484158ff4dafd219f7fd8542df23eb2a19eec58..a57762e914a29f15203673005bbbd8a0b49ea69d 100644
--- a/chrome/browser/extensions/chrome_extension_function_details.h
+++ b/chrome/browser/extensions/chrome_extension_function_details.h
@@ -62,6 +62,13 @@ class ChromeExtensionFunctionDetails {
   // 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_; }