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

Unified Diff: chrome/browser/android/cookies/cookies_fetcher.cc

Issue 1615773005: Rename first-party-only cookies to same-site cookies. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Tests. Created 4 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
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | chrome/browser/browsing_data/cookies_tree_model.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/android/cookies/cookies_fetcher.cc
diff --git a/chrome/browser/android/cookies/cookies_fetcher.cc b/chrome/browser/android/cookies/cookies_fetcher.cc
index c98ad30781764650d73ca6800dc91bbecd6bacda..abc62c6ec2c11735b7a351755f530e5745ed5972 100644
--- a/chrome/browser/android/cookies/cookies_fetcher.cc
+++ b/chrome/browser/android/cookies/cookies_fetcher.cc
@@ -88,7 +88,7 @@ void CookiesFetcher::OnCookiesFetchFinished(const net::CookieList& cookies) {
base::android::ConvertUTF8ToJavaString(env, i->Path()).obj(),
i->CreationDate().ToInternalValue(), i->ExpiryDate().ToInternalValue(),
i->LastAccessDate().ToInternalValue(), i->IsSecure(), i->IsHttpOnly(),
- i->IsFirstPartyOnly(), i->Priority());
+ i->IsSameSite(), i->Priority());
env->SetObjectArrayElement(joa.obj(), index++, java_cookie.obj());
}
@@ -110,7 +110,7 @@ void CookiesFetcher::RestoreCookies(JNIEnv* env,
int64_t last_access,
bool secure,
bool httponly,
- bool firstpartyonly,
+ bool same_site,
int priority) {
Profile* profile = ProfileManager::GetPrimaryUserProfile();
if (!profile->HasOffTheRecordProfile()) {
@@ -129,8 +129,8 @@ void CookiesFetcher::RestoreCookies(JNIEnv* env,
base::android::ConvertJavaStringToUTF8(env, path),
base::Time::FromInternalValue(creation),
base::Time::FromInternalValue(expiration),
- base::Time::FromInternalValue(last_access), secure, httponly,
- firstpartyonly, static_cast<net::CookiePriority>(priority));
+ base::Time::FromInternalValue(last_access), secure, httponly, same_site,
+ static_cast<net::CookiePriority>(priority));
// The rest must be done from the IO thread.
content::BrowserThread::PostTask(
@@ -166,9 +166,8 @@ void CookiesFetcher::RestoreToCookieJarInternal(
monster->SetCookieWithDetailsAsync(
cookie.Source(), cookie.Name(), cookie.Value(), cookie.Domain(),
cookie.Path(), cookie.ExpiryDate(), cookie.IsSecure(),
- cookie.IsHttpOnly(), cookie.IsFirstPartyOnly(),
- experimental_features_enabled, experimental_features_enabled,
- cookie.Priority(), cb);
+ cookie.IsHttpOnly(), cookie.IsSameSite(), experimental_features_enabled,
+ experimental_features_enabled, cookie.Priority(), cb);
}
// JNI functions
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | chrome/browser/browsing_data/cookies_tree_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698