Re: [w3c/IndexedDB] Implementation report (#213)

Wow, thanks for this awesome analysis! First pass of responses - I need to dig into the tests for the rest.

> Added details around garbage collection of IDBDatabase objects. (bug #25223)
> Not testable?

Not testable. :(

> Added [Exposed=(Window,Worker)] annotations to interfaces.
> Not testable? Or interfaces.{html,idl,worker.js}?

Tested by interfaces.worker.js

> Added forced flag to the steps to close a database connection, described the firing of a "close" event, and onclose. (bug #22540)
> Not testable?

Not testable, unfortunately - depends on implementation encountering an error or implementation-specific user action (E.g. clearing browsing data)

> Converted specification to a more algorithmic style, and define abstract types such as key more rigorously. (bug #17681)
> Not testable?

Not directly testable, although many edge cases discovered while doing this were covered with new tests.

> Clarified open request / connection queue processing. (bug #9, bug #78, bug #79, bug #81)
> Not testable?
> Or maybe idbfactory-open-request-success.html, idbfactory-open-request-error.html, and idbrequest-onupgradeneeded.htm?

open-request-queue.html, delete-request-queue.html

> Ensure event firing is done in the context of queued tasks. (bug #83)
> Not testable?

Implicitly tested throughout, the spec was just not precise here.

> Remove IDBEnvironment; expose globals with partial interface instead. (bug #94)
> Not testable?

Not testable - this is just a change in IDL style, not web-observable.

> Clarify when deleteDatabase() can fail. (bug #74)
> Not testable, perhaps? How to test whether it returns "an appropriate error"?

Not testable, unfortunately. It would require the implementation to hit an internal I/O error.

> Use [HTML]'s StructuredSerialize and StructuredDeserialize hooks. (bug #170)
> Not testable?

Not testable. 


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/IndexedDB/issues/213#issuecomment-335869039