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

Unified Diff: android_webview/browser/net/aw_cookie_store_wrapper.cc

Issue 2633663003: Implements strict secure cookies as the default behavior in //net (Closed)
Patch Set: Rebase on ToT Created 3 years, 11 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: android_webview/browser/net/aw_cookie_store_wrapper.cc
diff --git a/android_webview/browser/net/aw_cookie_store_wrapper.cc b/android_webview/browser/net/aw_cookie_store_wrapper.cc
index 466081b06a1c8fbc8f1b0d28b6e5d402dfac2abb..811e775cda493acc74d111e630a9240e784ec0b2 100644
--- a/android_webview/browser/net/aw_cookie_store_wrapper.cc
+++ b/android_webview/browser/net/aw_cookie_store_wrapper.cc
@@ -121,13 +121,11 @@ void SetCookieWithDetailsAsyncOnCookieThread(
bool secure,
bool http_only,
net::CookieSameSite same_site,
- bool enforce_strict_secure,
net::CookiePriority priority,
const net::CookieStore::SetCookiesCallback& callback) {
GetCookieStore()->SetCookieWithDetailsAsync(
url, name, value, domain, path, creation_time, expiration_time,
- last_access_time, secure, http_only, same_site, enforce_strict_secure,
- priority, callback);
+ last_access_time, secure, http_only, same_site, priority, callback);
}
void GetCookiesWithOptionsAsyncOnCookieThread(
@@ -222,14 +220,13 @@ void AwCookieStoreWrapper::SetCookieWithDetailsAsync(
bool secure,
bool http_only,
net::CookieSameSite same_site,
- bool enforce_strict_secure,
net::CookiePriority priority,
const SetCookiesCallback& callback) {
DCHECK(client_task_runner_->RunsTasksOnCurrentThread());
PostTaskToCookieStoreTaskRunner(
base::Bind(&SetCookieWithDetailsAsyncOnCookieThread, url, name, value,
domain, path, creation_time, expiration_time, last_access_time,
- secure, http_only, same_site, enforce_strict_secure, priority,
+ secure, http_only, same_site, priority,
CreateWrappedCallback<bool>(callback)));
}
« no previous file with comments | « android_webview/browser/net/aw_cookie_store_wrapper.h ('k') | android_webview/browser/net/aw_cookie_store_wrapper_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698