Re: [w3c/IndexedDB] Mark transactions inactive during key serialization (PR #479)

@asutherland approved this pull request.

Thank you so much for addressing this important spec problem and my apologies about the delays in my reviewing!

Restating:
- The existing "convert a value to a key" algorithm is renamed to "recursively convert a value to a key" and we introduce a new wrapper "convert a value to a key" that takes the optional transaction and wraps the underlying call to "recursively converting a value to a key" with marking the transaction inactive during the call and restoring it on the way out, making sure we conceptually are catching the exception and re-throwing it on the way out.

@SteveBeckerMSFT it sounded like you also wanted to take a review pass too?



-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/IndexedDB/pull/479#pullrequestreview-3829613936
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/IndexedDB/pull/479/review/3829613936@github.com>