Skip to content

Remove api.MessageEvent.origin.USVString_type from BCD#13136

Merged
foolip merged 1 commit intomdn:mainfrom
queengooborg:api/MessageEvent/origin/USVString_type
Oct 28, 2021
Merged

Remove api.MessageEvent.origin.USVString_type from BCD#13136
foolip merged 1 commit intomdn:mainfrom
queengooborg:api/MessageEvent/origin/USVString_type

Conversation

@queengooborg
Copy link
Contributor

This PR removes the USVString_type subfeature from api.MessageEvent.origin from BCD. This feature wouldn't be noticeable to end users.

@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Oct 27, 2021
@queengooborg queengooborg requested a review from foolip October 27, 2021 09:34
@foolip
Copy link
Contributor

foolip commented Oct 28, 2021

From https://github.com/mdn/browser-compat-data/pull/2086/files, we can see that this subfeature was added for a change in Firefox 55, which I think was mozilla/gecko-dev@29ca0de.

Note that the only change around origin there was replacing DOMString with USVString in the IDL. Since origin is readonly, that by itself actually has no effect at all. The way that web developers can notice that something is a USVString is they provide a JS string with lone surrogates as a USVString argument or attribute, which would cause those lone surrogates to be replaced.

@foolip
Copy link
Contributor

foolip commented Oct 28, 2021

@ddbeck FYI this could be tricky to explain in release notes. I would suggest keeping it simple, saying that this entry was about browser internals that web developers cannot observe.

@foolip foolip merged commit 574771e into mdn:main Oct 28, 2021
@queengooborg queengooborg deleted the api/MessageEvent/origin/USVString_type branch October 28, 2021 14:38
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Nov 2, 2021
ddbeck added a commit that referenced this pull request Nov 2, 2021
* Bump version to v4.0.9

* Set release date

* Add release note for #12795

* Add release note for #13006

* Add release note for #13136

* Add release note for #13076

* Add release note for #13216

* Add release note for #11924

* Add release note for #13143

* Undo list of one item

* Hoist known issues

* Add stats

* Fix compare link
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.

3 participants