[ios clean] Creates ToolbarWebState Mediator

ios/clean/chrome/browser/ui/toolbar/toolbar_coordinator.mm

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // ======                        New Architecture                         =====
 // =         This code is only used in the new iOS Chrome architecture.       =
 // ============================================================================
 
 #import "ios/clean/chrome/browser/ui/toolbar/toolbar_coordinator.h"
 
-#include "base/strings/sys_string_conversions.h"
 #import "ios/clean/chrome/browser/browser_coordinator+internal.h"
 #import "ios/clean/chrome/browser/ui/commands/toolbar_commands.h"
+#import "ios/clean/chrome/browser/ui/toolbar/toolbar_mediator.h"
 #import "ios/clean/chrome/browser/ui/toolbar/toolbar_view_controller.h"
 #import "ios/clean/chrome/browser/ui/tools/tools_coordinator.h"
 #import "ios/shared/chrome/browser/coordinator_context/coordinator_context.h"
-#include "ios/web/public/web_state/web_state.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 @interface ToolbarCoordinator ()<ToolbarCommands>
 @property(nonatomic, weak) ToolsCoordinator* toolsMenuCoordinator;
 @property(nonatomic, strong) ToolbarViewController* viewController;
+@property(nonatomic, strong) ToolbarMediator* mediator;
 @end
 
 @implementation ToolbarCoordinator
 @synthesize toolsMenuCoordinator = _toolsMenuCoordinator;
 @synthesize viewController = _viewController;
+@synthesize webState = _webState;
+@synthesize mediator = _mediator;
+
+- (instancetype)init {
+  if ((self = [super init])) {
+    _mediator = [[ToolbarMediator alloc] init];
+  }
+  return self;
+}
+
+- (void)setWebState:(web::WebState*)webState {
+  _webState = webState;
+  self.mediator.webState = self.webState;

[edchin, 2017/02/21 18:39:08]

Properly handle when the webState is nil or changes after the coordinator is
started.

[sczs, 2017/02/21 19:04:10]

Since this is a "parallel" coordinator I think it would be a good idea to leave
this as similar to:
https://codereview.chromium.org/2675763002/diff/80001/ios/clean/chrome/browse...
It would be weird to handle things differently and initialize them differently
from TabContainer. Also, since mediators are still on a Placeholder sorts of
stage, making any future changes would be easier to implement in both
Coordinators/Mediators.

+}
 
 - (void)start {
   self.viewController = [[ToolbarViewController alloc] init];
   self.viewController.toolbarCommandHandler = self;
+  self.mediator.consumer = self.viewController;

[edchin, 2017/02/21 18:39:08]

Create the Mediator here and set it's webState and consumer all at once.

 
   [self.context.baseViewController presentViewController:self.viewController
                                                 animated:self.context.animated
                                               completion:nil];
 }
 
-#pragma mark - CRWWebStateObserver
-
-- (void)webState:(web::WebState*)webState didLoadPageWithSuccess:(BOOL)success {
-  const GURL& pageURL = webState->GetVisibleURL();
-  [self.viewController
-      setCurrentPageText:base::SysUTF8ToNSString(pageURL.spec())];
-}
-
 #pragma mark - ToolbarCommands
 
 - (void)showToolsMenu {
   ToolsCoordinator* toolsCoordinator = [[ToolsCoordinator alloc] init];
   toolsCoordinator.toolbarCommandHandler = self;
   [self addChildCoordinator:toolsCoordinator];
   [toolsCoordinator start];
   self.toolsMenuCoordinator = toolsCoordinator;
 }
 
 - (void)closeToolsMenu {
   [self.toolsMenuCoordinator stop];
   [self removeChildCoordinator:self.toolsMenuCoordinator];
 }
 
 @end