RESOLVED FIXED Bug 59995
Port Mozilla's IndexedDB tests: cursor update updates indexes 
https://bugs.webkit.org/show_bug.cgi?id=59995
Summary Port Mozilla's IndexedDB tests: cursor update updates indexes
Mark Pilgrim (Google)
Reported 2011-05-02 18:15:16 PDT
Original test: http://mxr.mozilla.org/mozilla2.0/source/dom/indexedDB/test/test_cursor_update_updates_indexes.html?force=1 This is a port of a test from Mozilla's IndexedDB test suite. It checks that mutating records with a r/w cursor updates both unique and non-unique indexes on those records. WebKit passes this test.
Attachments
Patch (9.94 KB, patch)
2011-05-02 18:18 PDT, Mark Pilgrim (Google) 		no flags
DetailsFormatted DiffDiff
Patch (9.99 KB, patch)
2011-05-03 06:35 PDT, Mark Pilgrim (Google) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Mark Pilgrim (Google)
Comment 1 2011-05-02 18:18:00 PDT
Created attachment 92027 [details] Patch
Hans Wennborg
Comment 2 2011-05-03 03:09:52 PDT
Comment on attachment 92027 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=92027&action=review > LayoutTests/storage/indexeddb/mozilla/cursor-update-updates-indexes.html:44 > + START_DATA = evalAndLog("START_DATA = 'hi';"); Hmm, START_DATA and END_DATA don't feel like WebKit-style names. Looks good otherwise.
Mark Pilgrim (Google)
Comment 3 2011-05-03 06:35:03 PDT
Created attachment 92067 [details] Patch
Mark Pilgrim (Google)
Comment 4 2011-05-03 06:35:36 PDT
Updated variable names.
Hans Wennborg
Comment 5 2011-05-03 06:52:53 PDT
(In reply to comment #4) > Updated variable names. Unofficial r+.
WebKit Commit Bot
Comment 6 2011-05-03 12:11:49 PDT
Comment on attachment 92067 [details] Patch Clearing flags on attachment: 92067 Committed r85631: <http://trac.webkit.org/changeset/85631>
WebKit Commit Bot
Comment 7 2011-05-03 12:11:54 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.