session cookie not persisted on restart when applying browser update - follow up to 28379

[LaurenWags]

Description

Found while testing #28379

When session cookie is set and I go to apply browser update (used brave://settings/help), the session cookie is not retained after browser restart from that page.

Note - could be macOS specific as @goodov could not reproduce on Windows. Requesting others on @brave/qa-team to try various platforms to see if it reproduces for them.

Steps to Reproduce

1. Install slightly old beta version which includes this fix, but which will want to update. (I used 1.50.86)
2. Ensure "Continue where you left off" is enabled.
3. Open https://bayden.com/test/sessions/ and choose lightcyan from selector.
4. Ensure sessionCookie says color is: lightcyan.
5. Navigate to brave://settings/help and apply update via "Relaunch" button, browser should restart automatically.
6. Expect the session is restored and the website is still opened.
7. Navigate to test page and refresh the page (Ctrl/Cmd+R).
   --> Expect sessionCookie is still lightcyan. However sessionCookie field has no value instead

Actual result:

Expected result:

sessionCookie should be lightcyan after applying browser restart in this case

Reproduces how often:

easily on macOS, @goodov could not reproduce on Windows

Brave version (brave://version info)

Brave | 1.50.91 Chromium: 111.0.5563.64 (Official Build) beta (x86_64)
-- | --
Revision | c710e93d5b63b7095afe8c2c17df34408078439d-refs/branch-heads/5563@{#995}
OS | macOS Version 12.6.3 (Build 21G419)

Version/Channel Information:

• Can you reproduce this issue with the current release? n/a
• Can you reproduce this issue with the beta channel? yes
• Can you reproduce this issue with the nightly channel? likely

Other Additional Information:

• Does the issue resolve itself when disabling Brave Shields?
• Does the issue resolve itself when disabling Brave Rewards?
• Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

cc @rebron @goodov @brave/qa-team

[goodov]

Griffin change to disable this on MacOS is merged to staging: brave/brave-variations#557

@LaurenWags can you please test this to ensure the Griffin change resolves this on MacOS right now?

Cherry-pick to merge into production after confirmed the issue is fixed: brave/brave-variations#558

[goodov]

the underlying issue is in the update process: #25576

[LaurenWags]

@goodov I did the following:

1. Launched 1.50.86 as follows /Applications/Brave\ Browser\ Beta.app/Contents/MacOS/Brave\ Browser\ Beta --args --enable-logging=stderr --variations-server-url=https://variations.bravesoftware.com/seed - confirm version on brave://version, leave this page open
2. Ensured "Continue where you left off" is enabled.
3. Open https://bayden.com/test/sessions/ and choose lightcyan from selector.
4. Ensured sessionCookie says color is: lightcyan.
5. Navigate to brave://settings/help and apply update via "Relaunch" button, browser should restart automatically.
6. Expect the session is restored and the website is still opened.
7. Navigate to brave://version tab and confirm BraveCleanupSessionCookiesOnSessionRestore:Disabled shows on brave://version
8. Navigate to test page and refresh the page (Ctrl/Cmd+R).
9. Confirm sessionCookie is still lightcyan.

Step 1	Step 2	Step 4	Step 5	Step 7	Step 9

[MadhaviSeelam]

Followed STR from #29135 (comment) and couldn't reproduce in Win 11 x64

cc: @LaurenWags @goodov

2023-03-17_08h55_48.mp4

[goodov]

Followed STR from #29135 (comment) and couldn't reproduce in Win 11 x64

cc: @LaurenWags @goodov

yes, thank you! That's expected. We have this issue only on MacOS because of the broken manual update process.

[btlechowski]

Unable to reproduce on Linux

Brave	1.50.93 Chromium: 111.0.5563.64 (Official Build) beta (64-bit)
Revision	c710e93d5b63b7095afe8c2c17df34408078439d-refs/branch-heads/5563@{#995}
OS	Ubuntu 18.04 LTS

[kjozwiak]

@goodov I did the following:

1. Launched 1.50.86 as follows /Applications/Brave\ Browser\ Beta.app/Contents/MacOS/Brave\ Browser\ Beta --args --enable-logging=stderr --variations-server-url=https://variations.bravesoftware.com/seed - confirm version on brave://version, leave this page open
2. Ensured "Continue where you left off" is enabled.
3. Open https://bayden.com/test/sessions/ and choose lightcyan from selector.
4. Ensured sessionCookie says color is: lightcyan.
5. Navigate to brave://settings/help and apply update via "Relaunch" button, browser should restart automatically.
6. Expect the session is restored and the website is still opened.
7. Navigate to brave://version tab and confirm BraveCleanupSessionCookiesOnSessionRestore:Disabled shows on brave://version
8. Navigate to test page and refresh the page (Ctrl/Cmd+R).
9. Confirm sessionCookie is still lightcyan.

Verified the changes via Griffin using the template from above as per brave/brave-variations#557 (comment) before merging the change into production using brave/brave-variations#558.

[rtritto]

Any update?
After restarting Brave (v1.65.132), cookies are empty.