Refactored ExtensionUninstallDialog to take a NativeWindow instead of a Browser

chrome/browser/ui/browser_finder.cc

Index: chrome/browser/ui/browser_finder.cc
diff --git a/chrome/browser/ui/browser_finder.cc b/chrome/browser/ui/browser_finder.cc
index 92bb2f8432febfda4a37523f411682be6d4dfec4..99e8a1b34415359e80fd7d3f79d904401e2fbeb2 100644
--- a/chrome/browser/ui/browser_finder.cc
+++ b/chrome/browser/ui/browser_finder.cc
@@ -158,6 +158,12 @@ Browser* FindBrowserWithWindow(gfx::NativeWindow window) {
   return NULL;
 }
 
+gfx::NativeWindow FindWindowForBrowser(Browser* browser) {

[tapted, 2014/07/24 00:35:36]

I don't think this belongs here (it's not really a "find"). I think you can be
certain that for the sites currently passing a non-null browser, they are
guaranteed to have a non-null browser->window(), so I would just use
`browser->window()->GetNativeWindow()` instead of
`FindWindowForBrowser(browser)`.

[sashab, 2014/07/24 03:03:36]

Done.

+  if (browser && browser->window())
+    return browser->window()->GetNativeWindow();
+  return NULL;
+}
+
 Browser* FindBrowserWithWebContents(const WebContents* web_contents) {
   DCHECK(web_contents);
   for (TabContentsIterator it; !it.done(); it.Next()) {