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

Shared compression dictionaries #26257

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

tunetheweb
Copy link
Contributor

@tunetheweb tunetheweb commented Mar 19, 2025

Summary

Shared Compression Dictionary is a new standard to allow much smaller resource sizes by reusing either previous resources as dictionaries, or having a separate disctionary.

It is available using Brotli or the newer ZStd compression standards and has been available in Chrome since 130

This PR adds the changes to HTTP headers to BCD.

Test results and supporting details

https://chromestatus.com/feature/5124977788977152

Related issues

Equivalent Doc PR: mdn/content#38722

@tunetheweb
Copy link
Contributor Author

@pmeenan could you review this too?

@github-actions github-actions bot added data:http Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP size:l [PR only] 101-1000 LoC changed labels Mar 19, 2025
@pmeenan
Copy link
Contributor

pmeenan commented Mar 19, 2025

LGTM. The links will probably need to be updated once the RFC publishes (already approved, just waiting for the gears to turn).

@tunetheweb
Copy link
Contributor Author

tunetheweb commented Mar 19, 2025

Linting issue should be fixed once w3c/browser-specs#1737 is merged and released

@tunetheweb
Copy link
Contributor Author

Oh if there's a proper RFC then we should wait for that. Unless it's going to be a while? Do you know the RFC number yet?

@tunetheweb tunetheweb marked this pull request as draft March 19, 2025 15:16
@pmeenan
Copy link
Contributor

pmeenan commented Mar 19, 2025

Don't have a number yet but it may take a while because it is waiting for the shared brotli draft to get a RFC number first (also approved for publication). I'll see if I can nudge it but it's probably a few months out if history is any indicator

@github-actions github-actions bot added the data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML label Mar 19, 2025
@tunetheweb
Copy link
Contributor Author

The current version spec has been included in [email protected]. Waiting for it to be updated here, and then CI should (hopefully) pass, and then will mark this PR as ready for review.

@tunetheweb tunetheweb requested a review from caugner March 25, 2025 21:40
@tunetheweb tunetheweb marked this pull request as ready for review March 25, 2025 21:40
@tunetheweb
Copy link
Contributor Author

Woo hoo checks pass now so think this is ready for review.

@wbamberg I dunno if you wanna review this along with your kind offer to review mdn/content#38722? And if we should hold off merging these until they are both ready? What's the usual way here as sure this is a fairly common thing to do BCD and related content for the same feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML data:http Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP size:l [PR only] 101-1000 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants