Skip to content

Remove http.headers.Link.blocking #27145

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.

Sign up for GitHub

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

Merged
caugner merged 2 commits into from
Jun 28, 2025
Merged

Remove http.headers.Link.blocking #27145

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d6a5e60
Remove http.headers.Link.blocking
gsnedders Jun 25, 2025
dda32cf
Merge branch 'main' into patch-1
caugner Jun 27, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 0 additions & 36 deletions http/headers/Link.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,42 +32,6 @@
"deprecated": false
}
},
"blocking": {
"__compat": {
"spec_url": "https://html.spec.whatwg.org/multipage/semantics.html#attr-link-blocking",
"tags": [
"web-features:blocking-render"
],
"support": {
"chrome": {
"version_added": "105"
Copy link
Contributor

@caugner caugner Jun 25, 2025

Choose a reason for hiding this comment

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

To make this easier to comprehend, and based on chromium/chromium@1d8c45a#diff-4b938e3a1dedc77093c5a2e3471fb2cb1c9c89e953fefc2394d9b27b65f8c91d and chromium/chromium@1372f23, could we first update this statement in the next release, and delete the statement only in the following release:

Suggested change
"version_added": "105"
"version_added": "103",
"version_removed": "105"

/cc @ddbeck

Copy link
Contributor

@ddbeck ddbeck Jun 26, 2025

Choose a reason for hiding this comment

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

Yeah, it'd be nice for the blame to show this—so there's two commits for it. But it does meet the guidelines, so it's OK if it all happens in the same release.

Copy link
Contributor

@caugner caugner Jun 26, 2025

Choose a reason for hiding this comment

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

Thanks, Daniel. Let's do this then!

Copy link
Contributor

@caugner caugner Jun 26, 2025

Choose a reason for hiding this comment

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

@gsnedders I'll open a PR with the change above, and then we just need to rebase this one later.

Copy link
Contributor

@caugner caugner Jun 26, 2025

Choose a reason for hiding this comment

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

@ddbeck @gsnedders #27153

Copy link
Contributor

@caugner caugner Jun 27, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

That PR is merged, but I would prefer to have Chrome's removal in the published BCD version before preceeding with removing the feature altogether.

Copy link
Contributor Author

@gsnedders gsnedders Jun 27, 2025

Choose a reason for hiding this comment

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

Thanks for getting round to this before I had a moment!

},
"chrome_android": "mirror",
"edge": "mirror",
"firefox": {
"version_added": false,
"impl_url": "https://bugzil.la/1751383"
},
"firefox_android": "mirror",
"oculus": "mirror",
"opera": "mirror",
"opera_android": "mirror",
"safari": {
"version_added": false,
"impl_url": "https://webkit.org/b/267232"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror",
"webview_ios": "mirror"
},
"status": {
"experimental": true,
"standard_track": true,
"deprecated": false
}
}
},
"fetchpriority": {
"__compat": {
"spec_url": "https://html.spec.whatwg.org/multipage/semantics.html#attr-link-fetchpriority",
Expand Down