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

Unified Diff: chrome/test/data/extensions/api_test/cookies/api/tab.js

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: chrome/test/data/extensions/api_test/cookies/api/tab.js
diff --git a/chrome/test/data/extensions/api_test/cookies/api/tab.js b/chrome/test/data/extensions/api_test/cookies/api/tab.js
index cb5caf258be36da3c6454b2048ceb7c0847d9628..3bb210e60677b308c80535f24fe471811d1874c4 100644
--- a/chrome/test/data/extensions/api_test/cookies/api/tab.js
+++ b/chrome/test/data/extensions/api_test/cookies/api/tab.js
@@ -29,7 +29,7 @@ var TEST_DOMAIN_COOKIE = {
expirationDate: TEST_EXPIRATION_DATE
};
var TEST_SECURE_COOKIE = {
- url: TEST_URL5,
+ url: TEST_URL4,
name: 'SECRETCOOKIE',
value: 'foobar_password',
secure: true,
@@ -163,9 +163,9 @@ chrome.test.runTests([
function getSecureCookie() {
removeTestCookies();
chrome.cookies.set(TEST_SECURE_COOKIE, pass(function () {
- // Original URL doesn't work because scheme isn't secure.
+ // URL doesn't work because scheme isn't secure.
chrome.cookies.get(
- {url: TEST_SECURE_COOKIE.url, name: TEST_SECURE_COOKIE.name},
+ {url: TEST_URL5, name: TEST_SECURE_COOKIE.name},
pass(expectNullCookie));
// Path doesn't match.
chrome.cookies.get(
« no previous file with comments | « chrome/browser/net/chrome_network_delegate.cc ('k') | components/signin/core/browser/gaia_cookie_manager_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698