IndexedDB: setVersion batch 8
Created attachment 174519 [details] Patch
Josh, could you take a look? After this patch there are 19 layout more tests that have setVersion warnings in their expected.txt. There are a few more tests that have been disabled since before that warning was issued. There are also a few setVersion instances in the chrome browser tests that need to be taken care of.
Comment on attachment 174519 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=174519&action=review lgtm > LayoutTests/ChangeLog:18 > + Again awkward to call indexedDBTest all over the place. The prefix We can delete removeVendorPrefixes() whenever we want now, right? (i.e. after the last batch is done) > LayoutTests/ChangeLog:19 > + line's gotta go, and we could also pull a resetDatabase method out of How about simpleIndexedDBTest(upgradeNeededHandler, callback) that has onsuccess = { db.close(); callback(); } > LayoutTests/ChangeLog:39 > + * storage/indexeddb/resources/versionchangerequest-activedomobject.js: I don't know if this is exercising the same code as before (the test looks like it's watching for a specific regression), but it seems fine.
Comment on attachment 174519 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=174519&action=review >> LayoutTests/ChangeLog:18 >> + Again awkward to call indexedDBTest all over the place. The prefix > > We can delete removeVendorPrefixes() whenever we want now, right? (i.e. after the last batch is done) I think we should keep the prefix check in until 24 goes to stable in the off chance that we want to open any of the layout tests in 23. But we can remove its output via Quiet whenever. >> LayoutTests/ChangeLog:19 >> + line's gotta go, and we could also pull a resetDatabase method out of > > How about simpleIndexedDBTest(upgradeNeededHandler, callback) that has onsuccess = { db.close(); callback(); } That does seem promising.
Tony, could you review this?
Comment on attachment 174519 [details] Patch Clearing flags on attachment: 174519 Committed r134866: <http://trac.webkit.org/changeset/134866>
All reviewed patches have been landed. Closing bug.