Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(81)

Unified Diff: ios/chrome/browser/tabs/tab_model.mm

Issue 2807063006: [ios] Reduce access to private //ios/web API. (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/chrome/browser/tabs/tab_model.mm
diff --git a/ios/chrome/browser/tabs/tab_model.mm b/ios/chrome/browser/tabs/tab_model.mm
index 1bf1d713eb32753a8c48cd9bcea0b946f1ba71f4..7f4dfa849e88879c1a94e1d1e62ef8f05f90ec3a 100644
--- a/ios/chrome/browser/tabs/tab_model.mm
+++ b/ios/chrome/browser/tabs/tab_model.mm
@@ -686,10 +686,6 @@ void CleanCertificatePolicyCache(
#pragma mark - Private methods
- (SessionWindowIOS*)windowForSavingSession {
- // Background tabs will already have their state preserved, but not the
- // fg tab. Do it now.
- [self.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
@@ -720,9 +716,6 @@ void CleanCertificatePolicyCache(
int oldCount = _webStateList->count();
DCHECK_GE(oldCount, 0);
- if (persistState && self.currentTab)
- [self.currentTab recordStateInHistory];
-
web::WebState::CreateParams createParams(_browserState);
DeserializeWebStateList(
_webStateList.get(), window,

Powered by Google App Engine
This is Rietveld 408576698