Moved NavigationManagerImpl serialization out of CRWSessionController.

ios/chrome/browser/tabs/tab_model.mm

 // Copyright 2012 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/chrome/browser/tabs/tab_model.h"
 
 #include <list>
 #include <utility>
 #include <vector>
 
@@ skipping 48 matching lines @@
     @"kTabModelTabDeselectedNotification";
 NSString* const kTabModelNewTabWillOpenNotification =
     @"kTabModelNewTabWillOpenNotification";
 NSString* const kTabModelTabKey = @"tab";
 NSString* const kTabModelPageLoadSuccess = @"pageLoadSuccess";
 NSString* const kTabModelOpenInBackgroundKey = @"shouldOpenInBackground";
 
 namespace {
 
 // Updates CRWSessionCertificatePolicyManager's certificate policy cache.
-void UpdateCertificatePolicyCacheFromWebState(web::WebStateImpl* web_state) {
+void UpdateCertificatePolicyCacheFromWebState(web::WebState* web_state) {
   DCHECK([NSThread isMainThread]);
   DCHECK(web_state);
   scoped_refptr<web::CertificatePolicyCache> policy_cache =
       web::BrowserState::GetCertificatePolicyCache(
           web_state->GetBrowserState());
-  CRWSessionController* controller =
-      web_state->GetNavigationManagerImpl().GetSessionController();
+  // TODO(crbug.com/454984): Remove CRWSessionController usage once certificate
+  // policy manager is moved to NavigationManager.
+  CRWSessionController* controller = static_cast<web::WebStateImpl*>(web_state)
+                                         ->GetNavigationManagerImpl()
+                                         .GetSessionController();
   [[controller sessionCertificatePolicyManager]
       updateCertificatePolicyCache:policy_cache];
 }
 
 // Populates the certificate policy cache based on the current entries of the
 // given tabs.
 void RestoreCertificatePolicyCacheFromTabs(WebStateList* web_state_list) {
   DCHECK([NSThread isMainThread]);
   for (id<WebStateHandle> web_state in web_state_list) {
     UpdateCertificatePolicyCacheFromWebState(web_state.webStateImpl);
@@ skipping 170 matching lines @@
     // handles incognito browser states by filtering on profile, so it's
     // important to the backend code to always have a sync window delegate.
     if (!_browserState->IsOffTheRecord()) {
       // Set up the usage recorder before tabs are created.
       _tabUsageRecorder.reset(new TabUsageRecorder(self));
     }
     _syncedWindowDelegate.reset(new TabModelSyncedWindowDelegate(self));
 
     NSNotificationCenter* defaultCenter = [NSNotificationCenter defaultCenter];
     if (window) {
-      while (window.unclaimedSessions) {
-        std::unique_ptr<web::WebStateImpl> webState = [window nextSession];
+      for (CRWNavigationManagerSerialization* session in window.sessions) {
+        web::WebState::CreateParams params(_browserState, session);
+        std::unique_ptr<web::WebState> webState = web::WebState::Create(params);
         DCHECK_EQ(webState->GetBrowserState(), _browserState);
         // Restore the CertificatePolicyCache.
         UpdateCertificatePolicyCacheFromWebState(webState.get());
         // Create a new tab for each entry in the window. Don't send delegate
         // notifications for each restored tab, only when all done.
         base::scoped_nsobject<Tab> tab(
             [[Tab alloc] initWithWebState:std::move(webState) model:self]);
         [tab webController].usePlaceholderOverlay = YES;
         [tab fetchFavicon];
         [self addWebState:tab];
@@ skipping 42 matching lines @@
 }
 
 - (instancetype)init {
   NOTREACHED();
   return nil;
 }
 
 - (BOOL)restoreSessionWindow:(SessionWindowIOS*)window {
   DCHECK(_browserState);
   DCHECK(window);
-  if (!window.unclaimedSessions)
+  NSArray* sessions = window.sessions;
+  if (!sessions.count)
     return NO;
   size_t oldCount = self.count;
   size_t index = oldCount;
-  while (window.unclaimedSessions) {
-    std::unique_ptr<web::WebStateImpl> webState = [window nextSession];
+  for (CRWNavigationManagerSerialization* session in sessions) {
+    web::WebState::CreateParams params(_browserState, session);
+    std::unique_ptr<web::WebState> webState = web::WebState::Create(params);
     DCHECK_EQ(webState->GetBrowserState(), _browserState);
     Tab* tab = [self insertTabWithWebState:std::move(webState) atIndex:index++];
     tab.webController.usePlaceholderOverlay = YES;
     // Restore the CertificatePolicyCache. Note that after calling Pass()
     // |webState| is invalid, so we need to get the webstate from |tab|.
     UpdateCertificatePolicyCacheFromWebState(tab.webStateImpl);
   }
   DCHECK(self.count > oldCount);
   // If any tab was restored, the saved selected tab must be selected.
   if (self.count > oldCount) {
@@ skipping 501 matching lines @@
   // Background tabs will already have their state preserved, but not the
   // fg tab. Do it now.
   [_currentTab recordStateInHistory];
 
   // Build the array of sessions. Copy the session objects as the saving will
   // be done on a separate thread.
   // TODO(crbug.com/661986): This could get expensive especially since this
   // window may never be saved (if another call comes in before the delay).
   SessionWindowIOS* window = [[[SessionWindowIOS alloc] init] autorelease];
   for (Tab* tab in self) {
-    DCHECK(tab.webStateImpl);
-    std::unique_ptr<web::WebStateImpl> webStateCopy(
-        tab.webStateImpl->CopyForSessionWindow());
-    [window addSession:std::move(webStateCopy)];
+    web::WebState* webState = tab.webState;
+    DCHECK(webState);
+    [window addSerializedSession:webState->BuildSerializedNavigationManager()];
   }
   window.selectedIndex = [self indexOfTab:_currentTab];
   return window;
 }
 
 - (BOOL)isNTPTab:(Tab*)tab {
   std::string host = tab.url.host();
   return host == kChromeUINewTabHost || host == kChromeUIBookmarksHost;
 }
 
@@ skipping 199 matching lines @@
   web::NavigationManager::WebLoadParams params(URL);
   params.referrer = referrer;
   params.transition_type = ui::PAGE_TRANSITION_TYPED;
   [[tab webController] loadWithParams:params];
   [tab webController].webUsageEnabled = webUsageEnabled_;
   [self insertTab:tab atIndex:index];
   return tab;
 }
 
 @end