Skip to content

libservo: Extend SiteDataManager::clear_site_data to clear sessionStorage#41709

Merged
jdm merged 1 commit intoservo:mainfrom
janvarga:clear-site-data-session
Jan 6, 2026
Merged

libservo: Extend SiteDataManager::clear_site_data to clear sessionStorage#41709
jdm merged 1 commit intoservo:mainfrom
janvarga:clear-site-data-session

Conversation

@janvarga
Copy link
Copy Markdown
Member

@janvarga janvarga commented Jan 6, 2026

sessionStorage entries are identified via their associated origins and removed
across all browsing contexts. This brings sessionStorage in line with cookies,
which were already handled by the API.

Both public and private browsing contexts are included in the clearing
operation.

The necessary support has been added to the storage crate to clear WebStorage
data.

Testing: A new integration test has been added.

@janvarga janvarga marked this pull request as ready for review January 6, 2026 13:14
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 6, 2026
…torage

sessionStorage entries are identified via their associated origins and removed
across all browsing contexts. This brings sessionStorage in line with cookies,
which were already handled by the API.

Both public and private browsing contexts are included in the clearing
operation.

The necessary support has been added to the storage crate to clear WebStorage
data.

Signed-off-by: Jan Varga <[email protected]>
@janvarga janvarga force-pushed the clear-site-data-session branch from 7b843e2 to 24a0b5b Compare January 6, 2026 13:15
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 6, 2026
@jdm jdm added this pull request to the merge queue Jan 6, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 6, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 6, 2026
@servo-highfive servo-highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Jan 6, 2026
@janvarga
Copy link
Copy Markdown
Member Author

janvarga commented Jan 6, 2026

It seems it was just an intermittent failure?

@jdm jdm added this pull request to the merge queue Jan 6, 2026
@servo-highfive servo-highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Jan 6, 2026
Merged via the queue into servo:main with commit a2195e5 Jan 6, 2026
35 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants