Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5941)

Unified Diff: chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm

Issue 2163683005: Change ChooserController::Observer to ChooserController::View (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm b/chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm
index 52656f7ecbc29dfa92f9639a0b475cf790ffd56e..a72386c7501d613cba7dfa0a5e1e1df400ab3c46 100644
--- a/chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm
+++ b/chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm
@@ -40,13 +40,13 @@ const CGFloat kSeparatorHeight = 1.0f;
} // namespace
-class TableViewController : public ChooserController::Observer {
+class TableViewController : public ChooserController::View {
public:
TableViewController(ChooserController* chooser_controller,
NSTableView* table_view);
~TableViewController() override;
- // ChooserController::Observer:
+ // ChooserController::View:
void OnOptionsInitialized() override;
void OnOptionAdded(size_t index) override;
void OnOptionRemoved(size_t index) override;
@@ -67,11 +67,11 @@ TableViewController::TableViewController(ChooserController* chooser_controller,
: chooser_controller_(chooser_controller), table_view_(table_view) {
DCHECK(chooser_controller_);
DCHECK(table_view_);
- chooser_controller_->set_observer(this);
+ chooser_controller_->set_view(this);
}
TableViewController::~TableViewController() {
- chooser_controller_->set_observer(nullptr);
+ chooser_controller_->set_view(nullptr);
}
void TableViewController::OnOptionsInitialized() {
« no previous file with comments | « chrome/browser/ui/bluetooth/bluetooth_chooser_controller.cc ('k') | chrome/browser/ui/views/chooser_content_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698