Skip to content
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.

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

Remove IDBEnvironment mixin #12230

Merged
merged 1 commit into from Sep 4, 2021
Merged

Remove IDBEnvironment mixin #12230

merged 1 commit into from Sep 4, 2021

Conversation

Elchi3
Copy link
Member

@Elchi3 Elchi3 commented Aug 31, 2021

Yay, mixins.

Apparently the IDL was this:

interface mixin IDBEnvironment {
  [SameObject] readonly attribute IDBFactory indexedDB;
};

and it is now this:

partial interface mixin WindowOrWorkerGlobalScope {
  [SameObject] readonly attribute IDBFactory indexedDB;
};

So, BCD shouldn't have IDBEnvironment and also not WindowOrWorkerGlobalScope.indexeddb as that will soon be in a _globals folder. Proposing to just remove this file then.

@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Aug 31, 2021
@Elchi3 Elchi3 added the needs content update 📝 This PR needs a corresponding update to mdn/content to update the documentation label Aug 31, 2021
@Elchi3 Elchi3 added needs-release-note 📰 and removed needs content update 📝 This PR needs a corresponding update to mdn/content to update the documentation labels Aug 31, 2021
@queengooborg queengooborg merged commit b3a7f7a into mdn:main Sep 4, 2021
@Elchi3 Elchi3 deleted the rm-idbenv-mixin branch September 4, 2021 07:03
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Sep 10, 2021
ddbeck added a commit that referenced this pull request Sep 10, 2021
* Bump version to v4.0.3

* Hoist known issues

* Add release note for #12230

* Add release note for #12276

* Add release note for #12185

* Add release stats

* Add release date
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.

None yet

3 participants