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

Unified Diff: ios/shared/chrome/browser/ui/browser_list/browser_list_session_service_factory.mm

Issue 2900643003: [ios] Clean how WebUsageEnabled flag is restored for WebStateList. (Closed)
Patch Set: Created 3 years, 7 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/shared/chrome/browser/ui/browser_list/browser_list_session_service_factory.mm
diff --git a/ios/shared/chrome/browser/ui/browser_list/browser_list_session_service_factory.mm b/ios/shared/chrome/browser/ui/browser_list/browser_list_session_service_factory.mm
index f97f03bbfe532ca398a070a990af99b0941fb4a5..327681425f4c5f6070b6f1b7b1cfd4d8b7978314 100644
--- a/ios/shared/chrome/browser/ui/browser_list/browser_list_session_service_factory.mm
+++ b/ios/shared/chrome/browser/ui/browser_list/browser_list_session_service_factory.mm
@@ -37,6 +37,9 @@ std::unique_ptr<web::WebState> CreateWebState(
web::WebState::CreateParams(browser_state), session_storage);
web_state->GetSessionCertificatePolicyCache()->UpdateCertificatePolicyCache(
web::BrowserState::GetCertificatePolicyCache(browser_state));
+ // TODO(crbug.com/724282): Track whether web usage should be enabled for
+ // the deserialized WebStates.
+ web_state->SetWebUsageEnabled(true);
return web_state;
}
}

Powered by Google App Engine
This is Rietveld 408576698