Skip to content

Commit

Permalink
Add http.status.103 update manually
Browse files Browse the repository at this point in the history
  • Loading branch information
@Elchi3
Elchi3 committed Oct 25, 2023
1 parent 64c67e5 commit de13a92
Showing 1 changed file with 2 additions and 14 deletions.
16 changes: 2 additions & 14 deletions http/status.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -52,20 +52,8 @@
"chrome_android": "mirror",
"edge": "mirror",
"firefox": {
"version_added": "102",
"notes": "Supported in HTTP1.0 and later for preconnect.",
"flags": [
{
"type": "preference",
"name": "network.early-hints.enabled",
"value_to_set": "true"
},
{
"type": "preference",
"name": "network.early-hints.preconnect.enabled",
"value_to_set": "true"
}
]
"version_added": "120",
"notes": "Supported in HTTP1.0 and later for preconnect."

This comment has been minimized.

Sign in to view

Copy link
@hamishwillee

hamishwillee Oct 31, 2023 •
edited
Loading

Contributor

@Elchi3 network.early-hints.preconnect.enabled is now true by default, which is what gives us support for preconnect. HOwever network.early-hints.enabled is what gives us preload for this feature, and that is only enabled on early beta.

Do we need to capture that nuance - i.e. we've now lost network.early-hints.enabled.

I'm OK with this, just want to make sure you're sure!
We might reasonably instead indicate this as partial support, and have additional note "not supported for preload". It wouldn't be obvious to people that preload is expected to work from this.

This comment has been minimized.

Sign in to view

Copy link
@Elchi3

Elchi3 Nov 2, 2023

Author Member

Ah thanks! Actually, I guess we should be using sub features here.

  • http.status.103
  • http.status.103.preconnect
  • http.status.103.preload

I've done that in 7dda752

This comment has been minimized.

Sign in to view

Copy link
@hamishwillee

hamishwillee Nov 2, 2023 •
edited
Loading

Contributor

Yes, that's much better. But I added a pref for the preload.
},
"ie": {
"version_added": false
Expand Down

0 comments on commit de13a92

Please sign in to comment.