Commit 8efa8d0e authored by Maks Orlovich's avatar Maks Orlovich Committed by Commit Bot

Fix ChromeNetworkServiceBrowserTest.EncryptedCookies with in-development cookie changes

It's setting a 3rd-party-accessible cookie, so it must be secure

Bug: 1006816
Change-Id: I1a2511f9f682ddf103beba987d76e013d0b2750a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1860102Reviewed-by: default avatarLily Chen <chlily@chromium.org>
Commit-Queue: Maksim Orlovich <morlovich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#705581}
parent ef2de98a
......@@ -43,11 +43,12 @@ void SetCookie(
base::Time t = base::Time::Now();
net::CanonicalCookie cookie(kCookieName, kCookieValue, "www.test.com", "/", t,
t + base::TimeDelta::FromDays(1), base::Time(),
false, false, net::CookieSameSite::NO_RESTRICTION,
true /* secure */, false /* http-only*/,
net::CookieSameSite::NO_RESTRICTION,
net::COOKIE_PRIORITY_DEFAULT);
base::RunLoop run_loop;
cookie_manager->SetCanonicalCookie(
cookie, "http", net::CookieOptions(),
cookie, "https", net::CookieOptions(),
base::BindLambdaForTesting(
[&](net::CanonicalCookie::CookieInclusionStatus status) {
run_loop.Quit();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment