On Mon, Jan 23, 2012 at 5:17 PM, Joshua Bell <[email protected]> wrote: > On Mon, Jan 23, 2012 at 4:12 PM, Israel Hilerio <[email protected]> > wrote: >> >> In looking at the count method in IDBObjectStore and IDBIndex we noticed >> that its signature doesn't throw a TransactionInactiveError when the >> transaction being used is inactive. We would like to add this to the spec. > > Agreed. FWIW, this matches Chrome's behavior.
Same here. >> In addition, the index method in IDBObjectStore uses InvalidStateError to >> convey two different meanings: the object has been removed or deleted and >> the transaction being used finished. It seems that it would be better to >> separate these into: >> * InvalidStateError when the source object has been removed or deleted. >> * TransactionInactiveError when the transaction being used is inactive. >> >> What do you think? I can open a bug if we agree this is the desired >> behavior. > > > Did this come out of the discussion here: > > http://lists.w3.org/Archives/Public/public-webapps/2011OctDec/1589.html > > If so, the rationale for which exception type to use is included, although > no-one on the thread was deeply averse to the alternative. If it's a > different issue can give a more specific example? Right. I think InvalidStateErr is better, for the reason detailed in the above referenced email. I agree we're using the same exception for two error conditions, but I'm not terribly worried that this will make debugging harder for authors. But I don't feel strongly so if there's a good reason I'm ok with changing things. / Jonas
