Change ChooserController ownership model

chrome/browser/ui/views/browser_dialogs_views.cc

Index: chrome/browser/ui/views/browser_dialogs_views.cc
diff --git a/chrome/browser/ui/views/browser_dialogs_views.cc b/chrome/browser/ui/views/browser_dialogs_views.cc
index 952b90493470f7390f43114bc9e290e7e8e91736..8a0c833397add31a6143f656f826ebff6b1bfe9b 100644
--- a/chrome/browser/ui/views/browser_dialogs_views.cc
+++ b/chrome/browser/ui/views/browser_dialogs_views.cc
@@ -4,11 +4,14 @@
 
 #include "chrome/browser/ui/browser_dialogs.h"
 
+#include <memory>
+
 #include "chrome/browser/extensions/api/chrome_device_permissions_prompt.h"
 #include "chrome/browser/extensions/chrome_extension_chooser_dialog.h"
 #include "chrome/browser/extensions/extension_install_prompt.h"
 #include "chrome/browser/ui/login/login_handler.h"
 #include "chrome/browser/ui/views/new_task_manager_view.h"
+#include "components/chooser_controller/chooser_controller.h"

[msw, 2016/06/22 18:23:17]

nit: does chrome_extension_chooser_dialog.h's forward decl suffice here?

[juncai, 2016/06/22 22:41:20]

Tried commenting this #include, but got compile errors.

 
 // This file provides definitions of desktop browser dialog-creation methods for
 // all toolkit-views platforms other than Mac. It also provides the definitions
@@ -43,8 +46,8 @@ void ChromeDevicePermissionsPrompt::ShowDialog() {
 }
 
 void ChromeExtensionChooserDialog::ShowDialog(
-    ChooserController* chooser_controller) const {
-  ShowDialogImpl(chooser_controller);
+    std::unique_ptr<ChooserController> chooser_controller) const {
+  ShowDialogImpl(std::move(chooser_controller));
 }
 
 namespace chrome {