Re: [w3c/IndexedDB] Add getAllRecords() method and update getAll()/getAllKeys() to support direction option (PR #461)

@SteveBeckerMSFT commented on this pull request.



> @@ -1371,6 +1371,38 @@ To <dfn>convert a value to a key range</dfn> with
 
 </div>
 
+
+A <dfn>potentially valid key range</dfn> is a value that is a [=key/type|type of key=] or a [=key range=].
+However, [=convert a value to a key range|converting=] a [=potentially valid key range=] to a [=key range=] could throw an exception.
+
+NOTE:
+  For example, a value that is a [=BufferSource/detached|detached BufferSource=] is a [=potentially valid key range=] that will throw an exception when used with [=convert a value to a key range=].
+
+<div algorithm>
+
+  To determine when a value <dfn>is a potentially valid key range</dfn> with |value|, run these steps:

Re: having a unified type for "record snapshot" matches IDBCursorWithValue, which also provides attributes for key, primaryKey and value.

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

Message ID: <w3c/IndexedDB/pull/461/review/3045068604@github.com>