IndexedDB: Database metadata changes should be tied to transaction
Created attachment 112610 [details] Patch
Suggestions for additional tests welcome.
Chromium test that verifies the fix added at: http://codereview.chromium.org/8341092
Looks good to me.
Comment on attachment 112610 [details] Patch tony@, can you r/cq?
Comment on attachment 112610 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=112610&action=review > LayoutTests/storage/indexeddb/open-during-transaction.html:92 > +var successfullyParsed = true; Nit: You don't need to add this anymore. > LayoutTests/storage/indexeddb/version-change-abort.html:96 > +var successfullyParsed = true; Ditto
Created attachment 112883 [details] Patch
Comment on attachment 112883 [details] Patch thanks tony@ - it required slightly non-trivial tweak due to exception handling. One more try?
Comment on attachment 112883 [details] Patch Clearing flags on attachment: 112883 Committed r98749: <http://trac.webkit.org/changeset/98749>
All reviewed patches have been landed. Closing bug.