[iOS Clean] Created OverlayService.

ios/clean/chrome/browser/ui/web_contents/web_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.
 
 #import "ios/clean/chrome/browser/ui/web_contents/web_coordinator.h"
 
 #include "base/mac/foundation_util.h"
 #include "base/memory/ptr_util.h"
 #import "ios/clean/chrome/browser/ui/commands/context_menu_commands.h"
+#import "ios/clean/chrome/browser/ui/commands/overlay_commands.h"
 #import "ios/clean/chrome/browser/ui/context_menu/context_menu_context_impl.h"
 #import "ios/clean/chrome/browser/ui/context_menu/web_context_menu_coordinator.h"
+#import "ios/clean/chrome/browser/ui/overlays/overlay_queue.h"
 #import "ios/clean/chrome/browser/ui/web_contents/web_contents_mediator.h"
 #import "ios/clean/chrome/browser/ui/web_contents/web_contents_view_controller.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"
 #include "ios/web/public/navigation_manager.h"
 #include "ios/web/public/web_state/web_state.h"
 #import "ios/web/public/web_state/web_state_delegate_bridge.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
-@interface WebCoordinator ()<ContextMenuCommands, CRWWebStateDelegate> {
+@interface WebCoordinator ()<ContextMenuCommands,
+                             CRWWebStateDelegate,
+                             OverlayPresentationCommands> {
   std::unique_ptr<web::WebStateDelegateBridge> _webStateDelegate;
 }
 @property(nonatomic, strong) WebContentsViewController* viewController;
 @property(nonatomic, strong) WebContentsMediator* mediator;
+
+// Lazily creates a WebJavaScriptDialogPresenter and OverlayQueue for the
+// WebState.
+- (void)createDialogHelpers;
+
+// Starts the next overlay in |webState|'s OverlayQueue.
+- (void)startNextOverlay;
+
 @end
 
 @implementation WebCoordinator
 @synthesize webState = _webState;
 @synthesize viewController = _viewController;
 @synthesize mediator = _mediator;
 
 - (instancetype)init {
   if ((self = [super init])) {
     _mediator = [[WebContentsMediator alloc] init];
     _webStateDelegate = base::MakeUnique<web::WebStateDelegateBridge>(self);
   }
   return self;
 }
 
 - (void)setWebState:(web::WebState*)webState {
   // PLACEHOLDER: The web state delegate will be set by another object, and
   // this coordinator will not need to know the active web state.
   _webState = webState;
   self.webState->SetDelegate(_webStateDelegate.get());
 }
 
 - (void)start {
+  // Create the view controller and start it.
   self.viewController = [[WebContentsViewController alloc] init];
   self.mediator.consumer = self.viewController;
   self.mediator.webStateList = &self.browser->web_state_list();
   [super start];
+  // Register for overlay commands and start the next overlay.
+  [self.browser->dispatcher()
+      startDispatchingToTarget:self
+                   forProtocol:@protocol(OverlayPresentationCommands)];
+  [self createDialogHelpers];
+  [self startNextOverlay];
 }
 
 - (void)stop {
+  [self.browser->dispatcher() stopDispatchingToTarget:self];
   [super stop];
   [self.mediator disconnect];
 }
 
 - (void)childCoordinatorDidStart:(BrowserCoordinator*)childCoordinator {
   // Register to receive relevant ContextMenuCommands.
   if ([childCoordinator isKindOfClass:[WebContextMenuCoordinator class]]) {
     [self.browser->dispatcher()
         startDispatchingToTarget:self
                      forSelector:@selector(executeContextMenuScript:)];
@@ skipping 37 matching lines @@
     handleContextMenu:(const web::ContextMenuParams&)params {
   ContextMenuContextImpl* context =
       [[ContextMenuContextImpl alloc] initWithParams:params];
   WebContextMenuCoordinator* contextMenu =
       [[WebContextMenuCoordinator alloc] initWithContext:context];
   [self addChildCoordinator:contextMenu];
   [contextMenu start];
   return YES;
 }
 
+#pragma mark - OverlayPresentationCommands
+
+- (void)startNextOverlayForWebState:(web::WebState*)webState {
+  DCHECK_EQ(self.webState, webState);
+  [self startNextOverlay];
+}
+
+#pragma mark -
+
+- (void)createDialogHelpers {
+  id<OverlaySchedulerCommands> overlayDispatcher =
+      static_cast<id<OverlaySchedulerCommands>>(self.browser->dispatcher());
+  OverlayQueue::CreateForWebState(self.webState, overlayDispatcher);
+}
+
+- (void)startNextOverlay {
+  OverlayQueue* queue = OverlayQueue::FromWebState(self.webState);

[marq (ping after 24h), 2017/06/14 10:02:32]

This might belong in a mediator. 

(The general idea is that anything that's not hooking up objects and starting
them is outside the scope of a coordinator -- specifically we don't want
coordinator bloat to happen.)

[kkhorimoto, 2017/06/15 08:26:29]

This is no longer necessary.

+  if (queue->HasQueuedOverlays() && !queue->IsShowingOverlay())
+    queue->StartNextOverlay(self);
+}
+
 @end