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

Unified Diff: ios/clean/chrome/browser/ui/toolbar/toolbar_coordinator.mm

Issue 2935653002: [ios clean] Move dispatcher out of Browser.
Patch Set: Dispatcher out of Browser. Created 3 years, 6 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/toolbar/toolbar_coordinator.mm
diff --git a/ios/clean/chrome/browser/ui/toolbar/toolbar_coordinator.mm b/ios/clean/chrome/browser/ui/toolbar/toolbar_coordinator.mm
index 8b892dcccf5bb9934807b43fdf5cf0121ab28618..db8e24f3ce4f1615d8f3593347e475d0e1fa04d2 100644
--- a/ios/clean/chrome/browser/ui/toolbar/toolbar_coordinator.mm
+++ b/ios/clean/chrome/browser/ui/toolbar/toolbar_coordinator.mm
@@ -50,18 +50,19 @@
- (void)start {
self.viewController = [[ToolbarViewController alloc] init];
- CommandDispatcher* dispatcher = self.browser->dispatcher();
- [dispatcher startDispatchingToTarget:self
- forSelector:@selector(showToolsMenu)];
- [dispatcher startDispatchingToTarget:self
- forSelector:@selector(closeToolsMenu)];
- [dispatcher startDispatchingToTarget:self forSelector:@selector(goBack)];
- [dispatcher startDispatchingToTarget:self forSelector:@selector(goForward)];
- [dispatcher startDispatchingToTarget:self forSelector:@selector(reloadPage)];
- [dispatcher startDispatchingToTarget:self
- forSelector:@selector(stopLoadingPage)];
-
- self.viewController.dispatcher = static_cast<id>(self.browser->dispatcher());
+ [self.dispatcher startDispatchingToTarget:self
+ forSelector:@selector(showToolsMenu)];
+ [self.dispatcher startDispatchingToTarget:self
+ forSelector:@selector(closeToolsMenu)];
+ [self.dispatcher startDispatchingToTarget:self forSelector:@selector(goBack)];
+ [self.dispatcher startDispatchingToTarget:self
+ forSelector:@selector(goForward)];
+ [self.dispatcher startDispatchingToTarget:self
+ forSelector:@selector(reloadPage)];
+ [self.dispatcher startDispatchingToTarget:self
+ forSelector:@selector(stopLoadingPage)];
+
+ self.viewController.dispatcher = self.callableDispatcher;
self.mediator.consumer = self.viewController;
self.mediator.webStateList = &self.browser->web_state_list();
@@ -82,7 +83,7 @@
[self.browser->broadcaster()
removeObserver:self.mediator
forSelector:@selector(broadcastTabStripVisible:)];
- [self.browser->dispatcher() stopDispatchingToTarget:self];
+ [self.dispatcher stopDispatchingToTarget:self];
}
- (void)childCoordinatorDidStart:(BrowserCoordinator*)childCoordinator {

Powered by Google App Engine
This is Rietveld 408576698