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

Unified Diff: ios/clean/chrome/browser/ui/settings/settings_coordinator.mm

Issue 2779213003: [clean] Use CommandDispatcher to show/close Settings. (Closed)
Patch Set: Fix tests Created 3 years, 9 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: ios/clean/chrome/browser/ui/settings/settings_coordinator.mm
diff --git a/ios/clean/chrome/browser/ui/settings/settings_coordinator.mm b/ios/clean/chrome/browser/ui/settings/settings_coordinator.mm
index a3546fae93d1c8e5e38dbf9a0b8516e6b90ad8e8..4b6e6ed0af86ad87464003b7f688e1a321a4e08e 100644
--- a/ios/clean/chrome/browser/ui/settings/settings_coordinator.mm
+++ b/ios/clean/chrome/browser/ui/settings/settings_coordinator.mm
@@ -8,6 +8,7 @@
#import "ios/clean/chrome/browser/ui/commands/settings_commands.h"
#import "ios/shared/chrome/browser/coordinator_context/coordinator_context.h"
#import "ios/shared/chrome/browser/ui/browser_list/browser.h"
+#import "ios/shared/chrome/browser/ui/commands/command_dispatcher.h"
#import "ios/shared/chrome/browser/ui/coordinators/browser_coordinator+internal.h"
#if !defined(__has_feature) || !__has_feature(objc_arc)
@@ -19,7 +20,6 @@
@end
@implementation SettingsCoordinator
-@synthesize settingsCommandHandler = _settingsCommandHandler;
@synthesize viewController = _viewController;
#pragma mark - BrowserCoordinator
@@ -59,7 +59,7 @@
}
- (void)closeSettings {
- [self.settingsCommandHandler closeSettings];
+ [static_cast<id>(self.browser->dispatcher()) closeSettings];
}
@end
« no previous file with comments | « ios/clean/chrome/browser/ui/settings/settings_coordinator.h ('k') | ios/clean/chrome/browser/ui/tab_grid/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698