11394 2010-11-23 23:50:05 +0000 We should throw if continue() is called with a key <= the current position 2011-03-08 00:53:12 +0000 1 1 1 Unclassified WebAppsWG Indexed Database API unspecified PC All RESOLVED FIXED P2 normal --- 1 jonas jorlow jorlow mike public-webapps public-webapps-bugzilla oldest_to_newest 42740 0 jonas 2010-11-23 23:50:05 +0000 (The below assumes a forward iterating cursor, opposite applies for a reverse iterating cursor of course). The spec currently states that when calling IDBCursor.continue and passing in a key, the cursor moves to the first record which is greater than the current position, and greater than or equal to the passed in key. As we implemented this we realized that this makes the situation when someone passes in a key which is less than or equal to the current position very strange. The effect is that the key argument is ignored. However most likely this was done in error. Worst case, the author might have thought that the cursor would go back to the first entry whose key is greater than or equal to the passed in key, ignoring the current position. One example of buggy code would be displaying a sales-table with 10 entries from every employee name. If the code ends up passing in a key which is less than or equal to the passed in key, the code likely contains an error. So a better behavior would be to throw if the passed in key is less than or equal to the cursors current position. This makes it more clear that an error occurred, as well as helps a debugger detect the exception and display the current program state. 46307 1 jorlow 2011-03-04 04:02:08 +0000 What error should we throw? DATA_ERR? 46491 2 jorlow 2011-03-08 00:53:12 +0000 Jonas just fixed this.