Re: [EXTERNAL] Re: [blink-dev] Web-Facing Change PSA: Improved error reporting in IndexedDB for large value read failures

2024-10-16 Thread 'Abhishek Shanthkumar' via blink-dev
e aligning with Firefox is the best path forward here, although I don't feel strongly. I still think it's reasonable to do all this as a web-facing PSA instead of a full Intent to Ship, by the way. On Fri, Sep 20, 2024 at 2:37 AM 'Abhishek Shanthkumar' via blink-dev mail

Re: [EXTERNAL] Re: [blink-dev] Web-Facing Change PSA: Improved error reporting in IndexedDB for large value read failures

2024-09-23 Thread 'Abhishek Shanthkumar' via blink-dev
h they have not previously written to it, whereas "UnknownError" is used in cases where something goes wrong with underlying storage. So maybe aligning with Firefox is the best path forward here, although I don't feel strongly. I still think it's reasonable to do all this as

Re: [EXTERNAL] Re: [blink-dev] Web-Facing Change PSA: Improved error reporting in IndexedDB for large value read failures

2024-09-20 Thread 'Abhishek Shanthkumar' via blink-dev
something goes wrong with underlying storage. So maybe aligning with Firefox is the best path forward here, although I don't feel strongly. I still think it's reasonable to do all this as a web-facing PSA instead of a full Intent to Ship, by the way. On Fri, Sep 20, 2024 at 2:37 AM '

Re: [EXTERNAL] Re: [blink-dev] Web-Facing Change PSA: Improved error reporting in IndexedDB for large value read failures

2024-09-17 Thread 'Abhishek Shanthkumar' via blink-dev
Hi, Mike! The DOMException.name was "DataError" for all cases before this change. It remains the same for the "recoverable" cases even after this change. I'll add this to the Summary, thanks! This error is unique to Chromium because, yes, LevelDB is not used by other browsers. Thanks, Abhis

Re: [EXTERNAL] Re: [blink-dev] Web-Facing Change PSA: Improved error reporting in IndexedDB for large value read failures

2024-09-19 Thread 'Abhishek Shanthkumar' via blink-dev
A correction about the behavior of other browsers - it looks like Firefox too stores blobs and large values in files separate from the SQLite database. Deleting the files and then attempting to read the value raises a DOMException on the IDBRequest with these parameters: name: "UnknownError" mess

Re: [EXTERNAL] Re: [blink-dev] Web-Facing Change PSA: Improved error reporting in IndexedDB for large value read failures

2024-11-20 Thread 'Abhishek Shanthkumar' via blink-dev
f error, my read is that although the section at https://w3c.github.io/IndexedDB/#exceptions implies "NotFoundError" is a good choice, looking at actual usage in the spec, "NotFoundError" seems to be more about deterministic cases where the web developer is requesting something from