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

nolanlawson left a comment (w3c/IndexedDB#479)

Note I only added the minimal logic to mark the transaction inactive when a key is provided to `IDBObjectStore#add`/`put`. I did not:

- mark the transaction inactive anywhere else that we convert a key to a value, e.g. `indexedDB.cmp` or `IDBCursor#continue`, because this matches Firefox's implementation (and Chromium's as well – both browsers just throw `InternalError: too much recursion` / `RangeError: Maximum call stack size exceeded`
- refactor to reuse the existing logic from [`clone`](file:///home/nolan/workspace/IndexedDB/index.html#clone) – this seemed like overkill since the logic was only needed in one place.

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

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