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

Stop using DOMError #11

Closed
foolip opened this issue Jun 19, 2015 · 5 comments
Closed

Stop using DOMError #11

foolip opened this issue Jun 19, 2015 · 5 comments

Comments

@foolip
Copy link
Member

foolip commented Jun 19, 2015

DOMError is gone from https://dom.spec.whatwg.org/ and we're looking to get rid of it in Blink:
https://code.google.com/p/chromium/issues/detail?id=460725

CC @annevk (DOM) and @inexorabletash (IndexedDB has a similar situation)

@inexorabletash
Copy link
Member

In https://w3c.github.io/IndexedDB/ I just s/DOMError/DOMException/g but we haven't landed that yet. I think we're waiting on seeing usage numbers around DOMError that @foolip added, but maybe our "intent" thread just got forgotten.

@foolip
Copy link
Member Author

foolip commented Jul 17, 2015

Not forgotten, but waiting for this counter to reach stable in M45:
https://www.chromestatus.com/metrics/feature/timeline/popularity/816

These are also interesting, but not worrying compat-wise:
https://www.chromestatus.com/metrics/feature/timeline/popularity/817
https://www.chromestatus.com/metrics/feature/timeline/popularity/818

@inexorabletash
Copy link
Member

We just landed the s/DOMError/DOMException/g change Indexed DB in Chrome. Hopefully it sticks.

@Ms2ger
Copy link
Contributor

Ms2ger commented Aug 11, 2016

@mkruisselbrink could you fix?

@inexorabletash
Copy link
Member

In Blink we converted FileReader over to use DOMException for the error attribute back in July 2016 and the change stuck, so I think we can go ahead and update the spec.

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

No branches or pull requests

3 participants