[ios] [BVC refactor] Move -setActive: to coordinator

This CL moves the method -setActive: to the coordinator layer since
it deals with many child coordinators.

Change-Id: I8e61ece1380169ffc10e9e00bccaa68dbde7d4d4
Reviewed-on: https://chromium-review.googlesource.com/c/1355924Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Commit-Queue: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#613219}

ios/chrome/app/application_delegate/app_state.mm

@@ -42,6 +42,7 @@
 #import "ios/chrome/browser/ui/commands/application_commands.h"
 #import "ios/chrome/browser/ui/commands/browser_commands.h"
 #import "ios/chrome/browser/ui/commands/open_new_tab_command.h"
+#import "ios/chrome/browser/ui/main/browser_coordinator.h"
 #import "ios/chrome/browser/ui/main/browser_view_information.h"
 #import "ios/chrome/browser/ui/safe_mode/safe_mode_coordinator.h"
 #include "ios/chrome/browser/ui/util/ui_util.h"
@@ -402,7 +403,8 @@ initWithBrowserLauncher:(id<BrowserLauncher>)browserLauncher
   if ([_browserLauncher browserInitializationStage] >=
       INITIALIZATION_STAGE_FOREGROUND) {
     [[_browserLauncher browserViewInformation] haltAllTabs];
-    [_browserLauncher browserViewInformation].currentBVC.active = NO;
+    _browserLauncher.browserViewInformation.currentBrowserCoordinator.active =
+        NO;
   }
 
   // TODO(crbug.com/585700): remove this.

ios/chrome/app/main_controller.mm

@@ -885,8 +885,8 @@ enum class ShowTabSwitcherSnapshotResult {
   if (browsingDataRemover && browsingDataRemover->IsRemoving())
     return;
 
-  [self.mainBVC setActive:YES];
-  [self.otrBVC setActive:YES];
+  [self.mainBrowserCoordinator setActive:YES];
+  [self.incognitoBrowserCoordinator setActive:YES];
   [self.currentBVC setPrimary:YES];
 }
 
@@ -2163,8 +2163,8 @@ enum class ShowTabSwitcherSnapshotResult {
       // Disables browsing and purges web views.
       // Must be called only on the main thread.
       DCHECK([NSThread isMainThread]);
-      [self.mainBVC setActive:NO];
-      [self.otrBVC setActive:NO];
+      [self.mainBrowserCoordinator setActive:NO];
+      [self.incognitoBrowserCoordinator setActive:NO];
     }
 
     BrowsingDataRemoverFactory::GetForBrowserState(browserState)
@@ -2172,8 +2172,8 @@ enum class ShowTabSwitcherSnapshotResult {
                    // Activates browsing and enables web views.
                    // Must be called only on the main thread.
                    DCHECK([NSThread isMainThread]);
-                   [self.mainBVC setActive:YES];
-                   [self.otrBVC setActive:YES];
+                   [self.mainBrowserCoordinator setActive:YES];
+                   [self.incognitoBrowserCoordinator setActive:YES];
                    [self.currentBVC setPrimary:YES];
 
                    if (completionBlock)

ios/chrome/browser/ui/main/browser_coordinator.h

@@ -24,6 +24,12 @@
 // The model.
 @property(nonatomic, weak) TabModel* tabModel;
 
+// Activates/deactivates the object. This will enable/disable the ability for
+// this object to browse, and to have live UIWebViews associated with it. While
+// not active, the UI will not react to changes in the tab model, so generally
+// an inactive BVC should not be visible.
+@property(nonatomic, assign, getter=isActive) BOOL active;
+
 // Clears any presented state on BVC.
 - (void)clearPresentedStateWithCompletion:(ProceduralBlock)completion
                            dismissOmnibox:(BOOL)dismissOmnibox;

ios/chrome/browser/ui/main/browser_coordinator.mm

@@ -140,6 +140,16 @@
 
 #pragma mark - Public
 
+- (void)setActive:(BOOL)active {
+  DCHECK_EQ(_active, self.viewController.active);
+  if (_active == active) {
+    return;
+  }
+  _active = active;
+
+  self.viewController.active = active;
+}
+
 - (void)clearPresentedStateWithCompletion:(ProceduralBlock)completion
                            dismissOmnibox:(BOOL)dismissOmnibox {
   [self.passKitCoordinator stop];