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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/cookies/CookiesFetcher.java

Issue 1894213003: android: Fix CanonicalCookie same_site field (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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: chrome/android/java/src/org/chromium/chrome/browser/cookies/CookiesFetcher.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/cookies/CookiesFetcher.java b/chrome/android/java/src/org/chromium/chrome/browser/cookies/CookiesFetcher.java
index c5c42bf94afc5739a956e1a1b7466b1275832121..d453ad6a0313253ccb0b5b11e0df34a3bafcfc7e 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/cookies/CookiesFetcher.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/cookies/CookiesFetcher.java
@@ -174,7 +174,7 @@ public class CookiesFetcher {
nativeRestoreCookies(cookie.getUrl(), cookie.getName(), cookie.getValue(),
cookie.getDomain(), cookie.getPath(), cookie.getCreationDate(),
cookie.getExpirationDate(), cookie.getLastAccessDate(),
- cookie.isSecure(), cookie.isHttpOnly(), cookie.isSameSite(),
+ cookie.isSecure(), cookie.isHttpOnly(), cookie.getSameSite(),
cookie.getPriority());
}
}
@@ -219,7 +219,7 @@ public class CookiesFetcher {
@CalledByNative
private CanonicalCookie createCookie(String url, String name, String value, String domain,
String path, long creation, long expiration, long lastAccess, boolean secure,
- boolean httpOnly, boolean sameSite, int priority) {
+ boolean httpOnly, int sameSite, int priority) {
return new CanonicalCookie(url, name, value, domain, path, creation, expiration, lastAccess,
secure, httpOnly, sameSite, priority);
}
@@ -296,5 +296,5 @@ public class CookiesFetcher {
private native void nativePersistCookies(long nativeCookiesFetcher);
private static native void nativeRestoreCookies(String url, String name, String value,
String domain, String path, long creation, long expiration, long lastAccess,
- boolean secure, boolean httpOnly, boolean sameSite, int priority);
+ boolean secure, boolean httpOnly, int sameSite, int priority);
}

Powered by Google App Engine
This is Rietveld 408576698