Show extension uninstall dialog in browser window.

chrome/browser/extensions/api/management/chrome_management_api_delegate.cc

 // 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.
 
 #include "chrome/browser/extensions/api/management/chrome_management_api_delegate.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/bookmark_app_helper.h"
 #include "chrome/browser/extensions/chrome_requirements_checker.h"
 #include "chrome/browser/extensions/extension_service.h"
@@ skipping 59 matching lines @@
 
 class ManagementUninstallFunctionUninstallDialogDelegate
     : public extensions::ExtensionUninstallDialog::Delegate,
       public extensions::UninstallDialogDelegate {
  public:
   ManagementUninstallFunctionUninstallDialogDelegate(
       extensions::ManagementUninstallFunctionBase* function,
       const extensions::Extension* target_extension,
       bool show_programmatic_uninstall_ui)
       : function_(function) {
-    content::WebContents* web_contents = function->GetSenderWebContents();
+    Profile* profile = Profile::FromBrowserContext(function->browser_context());
+    Browser* browser = chrome::FindBrowserWithProfile(
+        profile, chrome::HOST_DESKTOP_TYPE_NATIVE);

[not at google - send to devlin, 2015/07/08 20:40:40]

Looks like you're not supposed to hard-code this enum, see:
https://sites.google.com/a/chromium.org/dev/developers/design-documents/aura/...

[wjywbs, 2015/07/08 23:01:21]

Done.

     extension_uninstall_dialog_.reset(
         extensions::ExtensionUninstallDialog::Create(
-            Profile::FromBrowserContext(function->browser_context()),
-            web_contents ? web_contents->GetTopLevelNativeWindow() : nullptr,
-            this));
+            profile, browser->window()->GetNativeWindow(), this));

[not at google - send to devlin, 2015/07/08 20:40:41]

I think it's better to at least try to show it in the WebContent's native
window, even though it's probably equivalent to the active window given it
requires a user action. Then fall back to the active window if necessary.

[wjywbs, 2015/07/08 23:01:21]

Done.

     if (show_programmatic_uninstall_ui) {
       extension_uninstall_dialog_->ConfirmUninstallByExtension(
           target_extension, function->extension(),
           extensions::UNINSTALL_REASON_MANAGEMENT_API);
     } else {
       extension_uninstall_dialog_->ConfirmUninstall(
           target_extension, extensions::UNINSTALL_REASON_MANAGEMENT_API);
     }
   }
   ~ManagementUninstallFunctionUninstallDialogDelegate() override {}
@@ skipping 214 matching lines @@
 
 GURL ChromeManagementAPIDelegate::GetIconURL(
     const extensions::Extension* extension,
     int icon_size,
     ExtensionIconSet::MatchType match,
     bool grayscale,
     bool* exists) const {
   return extensions::ExtensionIconSource::GetIconURL(extension, icon_size,
                                                      match, grayscale, exists);
 }