Commit 35e79e65 authored by Maks Orlovich's avatar Maks Orlovich Committed by Commit Bot

Pass proper options in ui_test_utils::GetCookies

So that cookies get looked up in 1st party, not 3rd party, context.
This should keep the tests:

IsolatedAppTest.CookieIsolation
IsolatedAppTest.SubresourceCookieIsolation
PlatformAppBrowserTest.Isolation
WebViewTest.CookieIsolation

working with upcoming cookie behavior changes.

Bug: 1006816
Change-Id: I3d366dcb484efa61045d6d339bb1e7d3c5062a08
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1844712Reviewed-by: default avatarLily Chen <chlily@chromium.org>
Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Commit-Queue: Maks Orlovich <morlovich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#703373}
parent 5b8b56fb
......@@ -517,10 +517,9 @@ void GetCookies(const GURL& url,
base::RunLoop loop;
auto* storage_partition =
contents->GetMainFrame()->GetProcess()->GetStoragePartition();
net::CookieOptions options;
net::CookieList cookie_list;
storage_partition->GetCookieManagerForBrowserProcess()->GetCookieList(
url, options,
url, net::CookieOptions::MakeAllInclusive(),
base::BindOnce(GetCookieCallback, loop.QuitClosure(), &cookie_list));
loop.Run();
......
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