[BUG] should get a non-session cookie fails on Chromium ToT

> playwright-internal@1.23.0-next ctest
> playwright test --config=tests/library/playwright.config.ts --project=chromium "tests\\library\\browsercontext-cookies.spec.ts:43" "--reporter=list"

Using executable at C:\Users\maxschmitt\Downloads\Win_x64_1006179_chrome-win\chrome-win\chrome.exe

Running 1 test using 1 worker

  ✘  [chromium] › library\browsercontext-cookies.spec.ts:43:1 › should get a non-session cookie (605ms)


  1) [chromium] › library\browsercontext-cookies.spec.ts:43:1 › should get a non-session cookie ====

    Error: expect(received).toBe(expected) // Object.is equality

    Expected: "username=John Doe"
    Received: ""

      50 |     return document.cookie;
      51 |   }, date);
    > 52 |   expect(documentCookie).toBe('username=John Doe');
         |                          ^
      53 |   const cookies = await context.cookies();
      54 |   expect(cookies.length).toBe(1);
      55 |   expect(cookies[0]).toEqual({

        at C:\Users\maxschmitt\development\playwright\tests\library\browsercontext-cookies.spec.ts:52:26


  1 failed
    [chromium] › library\browsercontext-cookies.spec.ts:43:1 › should get a non-session cookie =====
PS C:\Users\maxschmitt\development\playwright>

[arichiv]

Thanks for surfacing this, it ended up being a simple fix but would have been a disaster if it made it past dev to beta. Credit to @sbingler for figuring it out. https://chromium-review.googlesource.com/c/chromium/src/+/3702661

It’s on main now; I’m working to merge it into the M104 release as well and hope to have it in the weekly dev build.

[arichiv]

I was able to reproduce on tip-of-trunk with the same inconsistency on windows only. I’ll try to come up with a fix today and should be able to merge it into dev by end of week.