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

Unified Diff: components/cronet/ios/cronet_environment.mm

Issue 2684933009: [Cronet] Fix compilation and test errors for CookieStoreIOS. (Closed)
Patch Set: Address Eugene's suggestions. Created 3 years, 10 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
« no previous file with comments | « no previous file | ios/crnet/crnet_environment.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/ios/cronet_environment.mm
diff --git a/components/cronet/ios/cronet_environment.mm b/components/cronet/ios/cronet_environment.mm
index f9b90cecfeef6a9faf8c3b0e993c68a491789fa0..40fb6ac9bedadccaa129f0b50bd2ecff31a0567d 100644
--- a/components/cronet/ios/cronet_environment.mm
+++ b/components/cronet/ios/cronet_environment.mm
@@ -311,6 +311,11 @@ void CronetEnvironment::InitializeOnNetworkThread() {
context_builder.set_host_resolver(std::move(mapped_host_resolver));
+ // TODO(690969): This behavior matches previous behavior of CookieStoreIOS in
+ // CrNet, but should change to adhere to App's Cookie Accept Policy instead
+ // of changing it.
+ [[NSHTTPCookieStorage sharedHTTPCookieStorage]
+ setCookieAcceptPolicy:NSHTTPCookieAcceptPolicyAlways];
std::unique_ptr<net::CookieStore> cookie_store =
base::MakeUnique<net::CookieStoreIOS>(
[NSHTTPCookieStorage sharedHTTPCookieStorage]);
« no previous file with comments | « no previous file | ios/crnet/crnet_environment.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698