Throw DOMExceptions, not Errors (maybe?) about fakeindexeddb HOT 3 CLOSED

Thanks for the suggestion!

But I don't know if it's worth it... what would be gained, the Error.code property? That's already deprecated and I'm not sure if anyone uses it for IndexedDB. Anything else?

If there's a compelling use case, then yes please go for it. And also in src/test/web-platform-tests/wpt-env.js replace global.DOMException = Error; with importing that module to the global scope.

from fakeindexeddb.

My current use-case is code that looks like the following:

try {
  await doDbOperation();
} catch (err) {
  if (err instanceof DOMException && err.name === 'NotFoundError') {
    throw new MyErrorWrapper('message');
  } else {
    throw err;
  }
}

Cool, I'll see what I can do.

from fakeindexeddb.

Yeah that makes sense, I'd accept a PR :)

from fakeindexeddb.