Skip to content

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Jun 27, 2025

Based on #15006.

Summary

Adds the abort event on IDBDatabase.

Motivation

The IDBDatabase abort event is defined in the spec, and implemented in Chrome, Firefox, and Safari, so the data shouldn't
have been removed in the first place.

Important

While the same may apply to 50+ events that can bubble to Element, Document, and Window, we do not intend to add corresponding data to those three interfaces.

Test results and supporting details

The data was originally removed in #15006.

Related issues

Fixes: #15345

@github-actions github-actions bot added data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API size:m [PR only] 25-100 LoC changed labels Jun 27, 2025
Copy link
Contributor

github-actions bot commented Jun 27, 2025

Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs).

Copy link
Contributor

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

Since this is the PR that's going to appear in the commit history, can you add some text to the PR description summarizing that we're doing something exceptional here, why, and that we're not planning to extend this to other PRs? I'd like some kind of paper trail that doesn't require reading the entirety of at least one issue (and probably more).

@caugner
Copy link
Contributor Author

caugner commented Aug 19, 2025

can you add some text to the PR description summarizing that we're doing something exceptional here, why, and that we're not planning to extend this to other PRs?

@ddbeck Can you elaborate the "something exceptional" bit? I have added a Motivation section in the PR now:

The IDBDatabase event is defined in the spec, and implemented in Chrome, Firefox, and Safari, so the data shouldn't have been removed in the first place.

(Edit) I have now also added the following notecard:

Important

While the same may apply to 50+ events that can bubble to Element, Document, and Window, we do not intend to add corresponding data to those three interfaces.

@caugner caugner requested review from ddbeck and queengooborg and removed request for foolip August 19, 2025 18:22
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 size:m [PR only] 25-100 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No support data corresponding to IDBDatabase.onabort
3 participants