Re: [w3c/IndexedDB] Add IDBTransaction commit() method (#242)

Great cleanups!  They all look good to me.

> I'd be okay with making abort() throw if the state was_ committing_. Thoughts?

This sounds like the right choice to me.

> I was thinking about that too. But since `"complete"` is used as the event for "successfully committed" (vs. `"abort"` for "aborted") I didn't want to overload that already confusing term. Alternate ideas?

Given the cleanups you've made, I think my "Alternately..." sentence is nicely mooted.

-- 
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/pull/242#issuecomment-401654983

Received on Monday, 2 July 2018 02:28:17 UTC