Skip to content

Remove ConstrainLong and LongRange entries#9418

Merged
Elchi3 merged 1 commit intomdn:masterfrom
foolip:so-long-long
Mar 10, 2021
Merged

Remove ConstrainLong and LongRange entries#9418
Elchi3 merged 1 commit intomdn:masterfrom
foolip:so-long-long

Conversation

@foolip
Copy link
Contributor

@foolip foolip commented Mar 10, 2021

These are gone from the spec and MDN:
w3c/mediacapture-main#515
mdn/content#2995

@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Mar 10, 2021
@Elchi3 Elchi3 merged commit 0e140f6 into mdn:master Mar 10, 2021
@foolip foolip deleted the so-long-long branch March 10, 2021 15:29
@ddbeck
Copy link
Contributor

ddbeck commented Mar 11, 2021

@foolip @Elchi3 can you explain this in a little more detail? Was this never actually supported or was it dropped and now being removed as irrelevant? Something else?

I ask because I don't know what to write in the release notes for this one.

@Elchi3
Copy link
Member

Elchi3 commented Mar 11, 2021

My understanding of this PR is that these were renamed to
https://developer.mozilla.org/en-US/docs/Web/API/ConstrainULong
https://developer.mozilla.org/en-US/docs/Web/API/ULongRange

and the old names were dropped and now are now being removed as irrelevant here.

@ddbeck
Copy link
Contributor

ddbeck commented Mar 11, 2021

Very helpful, thank you!

ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Mar 11, 2021
@foolip
Copy link
Contributor Author

foolip commented Mar 11, 2021

It actually doesn't matter whether these types were ever used in implementations or not, because it amounts to changing a signed integer to an unsigned one. This has happened a lot (I've done a lot of such spec matching) but it's almost always the case that the values that are actually returned are unchanged. In JS there are only numbers, so a change like this is usually impossible to observe, unless some place that could return a negative number is also updated at the same time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants