Skip to content

Add Shared Compression Dictionary features#26257

Merged
caugner merged 18 commits intomdn:mainfrom
tunetheweb:shared-compression-dictionaries
Mar 28, 2025
Merged

Add Shared Compression Dictionary features#26257
caugner merged 18 commits intomdn:mainfrom
tunetheweb:shared-compression-dictionaries

Conversation

@tunetheweb
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

tunetheweb commented Mar 19, 2025

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

@tunetheweb
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

The current version spec has been included in web-specs@3.44.0. 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
Copy Markdown
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.

@wbamberg
Copy link
Copy Markdown
Contributor

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?

I'm probably not the best person to review BCD PRs, I'm afraid.

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.

We should try to be somewhat in sync, although IMO it's OK for things to be a little broken for a little while. It matters more that things aren't broken for a long while, which means, before we merge either of them, we should have a reasonable expectation that the other will get merged in a matter of days.

Once the MDN PR is ready to merge, let's check back on the state of this one :).

Copy link
Copy Markdown
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Overall LGTM, although I cannot easily verify the version numbers, except for the features introduced in Chrome 130.

image

Copy link
Copy Markdown
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Here are the Firefox impl_urls:

Co-authored-by: Claas Augner <495429+caugner@users.noreply.github.com>
@caugner caugner self-requested a review March 28, 2025 15:44
@caugner caugner changed the title Shared compression dictionaries Add Shared Compression Dictionary features Mar 28, 2025
@caugner caugner merged commit 50be839 into mdn:main Mar 28, 2025
5 checks passed
@mdn-bot mdn-bot mentioned this pull request Mar 28, 2025
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.

5 participants