Index: chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa.mm |
diff --git a/chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa.mm b/chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa.mm |
index c9885d542caae403b207ce0ff56b683a5cd58d4c..3affdb65869fd90a4ceff91e6ce832866382b157 100644 |
--- a/chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa.mm |
+++ b/chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa.mm |
@@ -13,17 +13,18 @@ |
#import "chrome/browser/ui/cocoa/constrained_window/constrained_window_custom_sheet.h" |
#import "chrome/browser/ui/cocoa/constrained_window/constrained_window_custom_window.h" |
#import "chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa_controller.h" |
+#include "components/chooser_controller/chooser_controller.h" |
#include "components/web_modal/web_contents_modal_dialog_manager.h" |
-ChooserDialogCocoa::ChooserDialogCocoa(content::WebContents* web_contents, |
- ChooserController* chooser_controller) |
- : web_contents_(web_contents), chooser_controller_(chooser_controller) { |
+ChooserDialogCocoa::ChooserDialogCocoa( |
+ content::WebContents* web_contents, |
+ std::unique_ptr<ChooserController> chooser_controller) |
+ : web_contents_(web_contents) { |
DCHECK(web_contents_); |
- DCHECK(chooser_controller_); |
- chooser_controller_->set_observer(this); |
+ DCHECK(chooser_controller); |
chooser_dialog_cocoa_controller_.reset([[ChooserDialogCocoaController alloc] |
initWithChooserDialogCocoa:this |
- chooserController:chooser_controller_]); |
+ chooserController:std::move(chooser_controller)]); |
} |
ChooserDialogCocoa::~ChooserDialogCocoa() {} |
@@ -33,21 +34,6 @@ void ChooserDialogCocoa::OnConstrainedWindowClosed( |
base::ThreadTaskRunnerHandle::Get()->DeleteSoon(FROM_HERE, this); |
} |
-void ChooserDialogCocoa::OnOptionsInitialized() { |
- if (chooser_dialog_cocoa_controller_) |
- [chooser_dialog_cocoa_controller_ onOptionsInitialized]; |
-} |
- |
-void ChooserDialogCocoa::OnOptionAdded(size_t index) { |
- if (chooser_dialog_cocoa_controller_) |
- [chooser_dialog_cocoa_controller_ onOptionAdded:index]; |
-} |
- |
-void ChooserDialogCocoa::OnOptionRemoved(size_t index) { |
- if (chooser_dialog_cocoa_controller_) |
- [chooser_dialog_cocoa_controller_ onOptionRemoved:index]; |
-} |
- |
void ChooserDialogCocoa::ShowDialog() { |
base::scoped_nsobject<NSWindow> window([[ConstrainedWindowCustomWindow alloc] |
initWithContentRect:[[chooser_dialog_cocoa_controller_ view] bounds]]); |
@@ -64,16 +50,17 @@ void ChooserDialogCocoa::Dismissed() { |
} |
void ChromeExtensionChooserDialog::ShowDialog( |
- ChooserController* chooser_controller) const { |
+ std::unique_ptr<ChooserController> chooser_controller) const { |
if (chrome::ToolkitViewsWebUIDialogsEnabled()) |
- return ChromeExtensionChooserDialog::ShowDialogImpl(chooser_controller); |
+ return ChromeExtensionChooserDialog::ShowDialogImpl( |
+ std::move(chooser_controller)); |
web_modal::WebContentsModalDialogManager* manager = |
web_modal::WebContentsModalDialogManager::FromWebContents(web_contents_); |
if (manager) { |
// These objects will delete themselves when the dialog closes. |
ChooserDialogCocoa* chooser_dialog = |
- new ChooserDialogCocoa(web_contents_, chooser_controller); |
+ new ChooserDialogCocoa(web_contents_, std::move(chooser_controller)); |
chooser_dialog->ShowDialog(); |
} |
} |