Revert "[iOS] Plumb Browser into RecentTabsViewController"

This reverts commit d3273ac4.

Reason for revert: Causing crashes on (internal) iphone-device and ipad-device. Might be a Debug vs Release issue, rather than a device vs simulator issue.

https://ci.chromium.org/p/chrome/builders/ci/iphone-device/7243
https://ci.chromium.org/p/chrome/builders/ci/ipad-device/7808

Original change's description:
> [iOS] Plumb Browser into RecentTabsViewController
> 
> Changes RecentTabsViewController to accept a Browser instead of separate
> WebStateList and BrowserState inputs. This will make future refactoring
> more straightforward.
> 
> (There remains a great deal of work to do to fully refactor Recent Tabs
> to properly factor responsibilities into a mediator and coordinator).
> 
> Bug: 845192
> Change-Id: I068b0a343752dba602d2a2226cd0bcf327873446
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078452
> Commit-Queue: Mark Cogan <marq@chromium.org>
> Reviewed-by: Gauthier Ambard <gambard@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#745456}

TBR=marq@chromium.org,gambard@chromium.org

Change-Id: I7119af082ce06b110638bda6ba6782f4382c1f16
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 845192
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2079468Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Commit-Queue: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#745562}

ios/chrome/browser/ui/recent_tabs/BUILD.gn

@@ -71,7 +71,6 @@ source_set("recent_tabs_ui") {
     "//components/sync",
     "//ios/chrome/app/strings",
     "//ios/chrome/browser/browser_state",
-    "//ios/chrome/browser/main:public",
     "//ios/chrome/browser/metrics:metrics_internal",
     "//ios/chrome/browser/sessions",
     "//ios/chrome/browser/sessions:serialisation",

ios/chrome/browser/ui/recent_tabs/recent_tabs_coordinator.mm

@@ -47,13 +47,14 @@
   // Initialize and configure RecentTabsTableViewController.
   RecentTabsTableViewController* recentTabsTableViewController =
       [[RecentTabsTableViewController alloc] init];
-  recentTabsTableViewController.browser = self.browser;
+  recentTabsTableViewController.browserState = self.browserState;
   recentTabsTableViewController.loadStrategy = self.loadStrategy;
   CommandDispatcher* dispatcher = self.browser->GetCommandDispatcher();
   id<ApplicationCommands> handler =
       HandlerForProtocol(dispatcher, ApplicationCommands);
   recentTabsTableViewController.handler = handler;
   recentTabsTableViewController.presentationDelegate = self;
+  recentTabsTableViewController.webStateList = self.browser->GetWebStateList();
 
   // Adds the "Done" button and hooks it up to |stop|.
   UIBarButtonItem* dismissButton = [[UIBarButtonItem alloc]

ios/chrome/browser/ui/recent_tabs/recent_tabs_table_view_controller.h

@@ -9,8 +9,9 @@
 #import "ios/chrome/browser/ui/table_view/chrome_table_view_controller.h"
 #include "ui/base/window_open_disposition.h"
 
-class Browser;
+class ChromeBrowserState;
 enum class UrlLoadStrategy;
+class WebStateList;
 
 @protocol ApplicationCommands;
 @protocol RecentTabsTableViewControllerDelegate;
@@ -20,9 +21,8 @@ enum class UrlLoadStrategy;
 @interface RecentTabsTableViewController
     : ChromeTableViewController <RecentTabsConsumer,
                                  UIAdaptivePresentationControllerDelegate>
-// The Browser for the tabs being restored. It's an error to pass a nullptr
-// Browser.
-@property(nonatomic, assign) Browser* browser;
+// The coordinator's BrowserState.
+@property(nonatomic, assign) ChromeBrowserState* browserState;
 // The command handler used by this ViewController.
 @property(nonatomic, weak) id<ApplicationCommands> handler;
 // Opaque instructions on how to open urls.
@@ -31,6 +31,8 @@ enum class UrlLoadStrategy;
 @property(nonatomic, assign) WindowOpenDisposition restoredTabDisposition;
 // RecentTabsTableViewControllerDelegate delegate.
 @property(nonatomic, weak) id<RecentTabsTableViewControllerDelegate> delegate;
+// WebStateList for tabs restored by this object.
+@property(nonatomic, assign) WebStateList* webStateList;
 // Whether the updates of the RecentTabs should be ignored. Setting this to NO
 // would trigger a reload of the TableView.
 @property(nonatomic, assign) BOOL preventUpdates;

ios/chrome/browser/ui/recent_tabs/recent_tabs_table_view_controller.mm

@@ -16,7 +16,6 @@
 #include "components/sync_sessions/open_tabs_ui_delegate.h"
 #include "components/sync_sessions/session_sync_service.h"
 #include "ios/chrome/browser/browser_state/chrome_browser_state.h"
-#import "ios/chrome/browser/main/browser.h"
 #import "ios/chrome/browser/metrics/new_tab_page_uma.h"
 #include "ios/chrome/browser/sessions/session_util.h"
 #include "ios/chrome/browser/sessions/tab_restore_service_delegate_impl_ios.h"
@@ -118,13 +117,8 @@ const int kRecentlyClosedTabsSectionIndex = 0;
 // Handles displaying the context menu for all form factors.
 @property(nonatomic, strong) ContextMenuCoordinator* contextMenuCoordinator;
 @property(nonatomic, strong) SigninPromoViewMediator* signinPromoViewMediator;
-// The browser state used for many operations, derived from the one provided by
-// |self.browser|.
-@property(nonatomic, readonly) ChromeBrowserState* browserState;
 // YES if this ViewController is being presented on incognito mode.
-@property(nonatomic, readonly, getter=isIncognito) BOOL incognito;
-// Convenience getter for |self.browser|'s WebStateList
-@property(nonatomic, readonly) WebStateList* webStateList;
+@property(nonatomic, assign, getter=isIncognito) BOOL incognito;
 @end
 
 @implementation RecentTabsTableViewController : ChromeTableViewController
@@ -180,20 +174,15 @@ const int kRecentlyClosedTabsSectionIndex = 0;
 
 #pragma mark - Setters & Getters
 
-- (void)setBrowser:(Browser*)browser {
-  DCHECK(browser);
-  _browser = browser;
-  ChromeBrowserState* browserState = browser->GetBrowserState();
-  // Some RecentTabs services depend on objects not present in the OffTheRecord
-  // BrowserState, in order to prevent crashes set |_browserState| to
-  // |browserState|->OriginalChromeBrowserState. While doing this check if
-  // incognito or not so that pages are loaded accordingly.
-  _browserState = browserState->GetOriginalChromeBrowserState();
-  _incognito = browserState->IsOffTheRecord();
-}
-
-- (WebStateList*)webStateList {
-  return self.browser->GetWebStateList();
+// Some RecentTabs services depend on objects not present in the OffTheRecord
+// BrowserState, in order to prevent crashes set |_browserState| to
+// |browserState|->OriginalChromeBrowserState. While doing this check if
+// incognito or not so that pages are loaded accordingly.
+- (void)setBrowserState:(ChromeBrowserState*)browserState {
+  if (browserState) {
+    _browserState = browserState->GetOriginalChromeBrowserState();
+    _incognito = browserState->IsOffTheRecord();
+  }
 }
 
 - (void)setPreventUpdates:(BOOL)preventUpdates {

ios/chrome/browser/ui/tab_grid/tab_grid_coordinator.mm

@@ -182,7 +182,8 @@
 
   // TODO(crbug.com/845192) : Remove RecentTabsTableViewController dependency on
   // ChromeBrowserState so that we don't need to expose the view controller.
-  baseViewController.remoteTabsViewController.browser = self.regularBrowser;
+  baseViewController.remoteTabsViewController.browserState =
+      regularBrowserState;
   self.remoteTabsMediator = [[RecentTabsMediator alloc] init];
   self.remoteTabsMediator.browserState = regularBrowserState;
   self.remoteTabsMediator.consumer = baseViewController.remoteTabsConsumer;
@@ -200,6 +201,8 @@
       UrlLoadStrategy::ALWAYS_NEW_FOREGROUND_TAB;
   baseViewController.remoteTabsViewController.restoredTabDisposition =
       WindowOpenDisposition::NEW_FOREGROUND_TAB;
+  baseViewController.remoteTabsViewController.webStateList =
+      regularWebStateList;
   baseViewController.remoteTabsViewController.presentationDelegate = self;
 
   if (!base::FeatureList::IsEnabled(kContainedBVC)) {

ios/chrome/browser/ui/tab_grid/tab_grid_coordinator_unittest.mm

@@ -7,11 +7,9 @@
 #import <UIKit/UIKit.h>
 
 #import "base/test/ios/wait_util.h"
-#import "ios/chrome/browser/main/test_browser.h"
 #import "ios/chrome/browser/ui/commands/browsing_data_commands.h"
 #import "ios/chrome/browser/ui/tab_grid/tab_switcher.h"
 #import "ios/chrome/test/block_cleanup_test.h"
-#include "ios/web/public/test/web_task_environment.h"
 #include "testing/gtest_mac.h"
 #include "third_party/ocmock/OCMock/OCMock.h"
 
@@ -41,7 +39,6 @@ namespace {
 class TabGridCoordinatorTest : public BlockCleanupTest {
  public:
   TabGridCoordinatorTest() {
-    browser_ = std::make_unique<TestBrowser>();
     UIWindow* window = [UIApplication sharedApplication].keyWindow;
     coordinator_ = [[TabGridCoordinator alloc]
                      initWithWindow:window
@@ -50,7 +47,6 @@ class TabGridCoordinatorTest : public BlockCleanupTest {
         browsingDataCommandEndpoint:OCMProtocolMock(
                                         @protocol(BrowsingDataCommands))];
     coordinator_.animationsDisabledForTesting = YES;
-    coordinator_.regularBrowser = browser_.get();
     // TabGirdCoordinator will make its view controller the root, so stash the
     // original root view controller before starting |coordinator_|.
     original_root_view_controller_ =
@@ -76,14 +72,9 @@ class TabGridCoordinatorTest : public BlockCleanupTest {
           original_root_view_controller_;
       original_root_view_controller_ = nil;
     }
-    [coordinator_ stop];
   }
 
  protected:
-  web::WebTaskEnvironment task_environment_;
-  // Browser for the coordinator.
-  std::unique_ptr<Browser> browser_;
-
   // The TabGridCoordinator that is under test.  The test fixture sets
   // this VC as the root VC for the window.
   TabGridCoordinator* coordinator_;
@@ -215,6 +206,7 @@ TEST_F(TabGridCoordinatorTest, CompletionHandlers) {
 // Test that the tab grid coordinator sizes its view controller to the window.
 TEST_F(TabGridCoordinatorTest, SizeTabGridCoordinatorViewController) {
   CGRect rect = [UIScreen mainScreen].bounds;
+  [coordinator_ start];
   EXPECT_TRUE(
       CGRectEqualToRect(rect, coordinator_.baseViewController.view.frame));
 }