Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changed storage.session quota limit from 1MB to 10MB. #24853

Merged
merged 1 commit into from
Feb 25, 2023

Conversation

jdatapple
Copy link
Contributor

@jdatapple jdatapple commented Feb 24, 2023

Description

Changed the mention of a 1MB storage quota limit for sessions to a 10MB quota limit.

Motivation

This change is based on a recommendation on a BCD PR.

Additional details

Reference for Chrome bug that bumped the limit to 10 MB:
https://bugs.chromium.org/p/chromium/issues/detail?id=1385167

Related issues and pull requests

Related to #18997.

@jdatapple jdatapple requested a review from a team as a code owner February 24, 2023 22:41
@jdatapple jdatapple requested review from willdurand and removed request for a team February 24, 2023 22:41
@github-actions github-actions bot added Content:Other Any docs not covered by another "Content:" label Content:WebExt WebExtensions docs labels Feb 24, 2023
@jdatapple jdatapple changed the title Changed quote limit from 1MB to 10MB. Changed storage.session quota limit from 1MB to 10MB. Feb 24, 2023
Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! And congratulations on your first merged MDN/content PR. Welcome, again, to the team. 🥳

@estelle estelle merged commit 3180c71 into mdn:main Feb 25, 2023
@jdatapple
Copy link
Contributor Author

Aww, thank you @estelle 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Other Any docs not covered by another "Content:" label Content:WebExt WebExtensions docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants