IndexedDB: Don't wedge if page reloads with pending upgradeneeded
Created attachment 166571 [details] Patch
Created attachment 166572 [details] Patch
Created attachment 166582 [details] Patch
Josh, could you look at this?
Created attachment 166586 [details] Patch
Lgtm
Tony, could you review this?
Comment on attachment 166586 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=166586&action=review > LayoutTests/storage/indexeddb/resources/dont-wedge.js:26 > + else > + debug("In content_shell this deleteDatabase request is blocked temporarily"); Nit: I would make this comment more general. Rather than name content_shell, maybe say in a multiprocess implementation, this deleteDatabase request can be blocked temporarily. Someone may want to run this test in another browser or in a few years, we may be using something other than content_shell.
Comment on attachment 166586 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=166586&action=review >> LayoutTests/storage/indexeddb/resources/dont-wedge.js:26 >> + debug("In content_shell this deleteDatabase request is blocked temporarily"); > > Nit: I would make this comment more general. Rather than name content_shell, maybe say in a multiprocess implementation, this deleteDatabase request can be blocked temporarily. Someone may want to run this test in another browser or in a few years, we may be using something other than content_shell. SG, thanks for the review.
Created attachment 166723 [details] Patch for landing
Comment on attachment 166723 [details] Patch for landing Clearing flags on attachment: 166723 Committed r130199: <http://trac.webkit.org/changeset/130199>
All reviewed patches have been landed. Closing bug.