RESOLVED FIXED 98091
IndexedDB: Don't wedge if page reloads with pending upgradeneeded 
https://bugs.webkit.org/show_bug.cgi?id=98091
Summary IndexedDB: Don't wedge if page reloads with pending upgradeneeded
David Grogan
Reported 2012-10-01 16:44:39 PDT
IndexedDB: Don't wedge if page reloads with pending upgradeneeded
Attachments
Patch (6.81 KB, patch)
2012-10-01 16:51 PDT, David Grogan 		no flags
DetailsFormatted DiffDiff
Patch (6.90 KB, patch)
2012-10-01 16:53 PDT, David Grogan 		no flags
DetailsFormatted DiffDiff
Patch (6.92 KB, patch)
2012-10-01 18:01 PDT, David Grogan 		no flags
DetailsFormatted DiffDiff
Patch (7.01 KB, patch)
2012-10-01 18:13 PDT, David Grogan 		no flags
DetailsFormatted DiffDiff
Patch for landing (7.17 KB, patch)
2012-10-02 11:43 PDT, David Grogan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
David Grogan
Comment 1 2012-10-01 16:51:33 PDT
Created attachment 166571 [details] Patch
David Grogan
Comment 2 2012-10-01 16:53:40 PDT
Created attachment 166572 [details] Patch
David Grogan
Comment 3 2012-10-01 18:01:41 PDT
Created attachment 166582 [details] Patch
David Grogan
Comment 4 2012-10-01 18:03:33 PDT
Josh, could you look at this?
David Grogan
Comment 5 2012-10-01 18:13:15 PDT
Created attachment 166586 [details] Patch
Joshua Bell
Comment 6 2012-10-01 18:22:24 PDT
Lgtm
David Grogan
Comment 7 2012-10-01 18:45:13 PDT
Tony, could you review this?
Tony Chang
Comment 8 2012-10-02 09:53:30 PDT
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.
David Grogan
Comment 9 2012-10-02 11:24:49 PDT
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.
David Grogan
Comment 10 2012-10-02 11:43:26 PDT
Created attachment 166723 [details] Patch for landing
WebKit Review Bot
Comment 11 2012-10-02 13:09:04 PDT
Comment on attachment 166723 [details] Patch for landing Clearing flags on attachment: 166723 Committed r130199: <http://trac.webkit.org/changeset/130199>
WebKit Review Bot
Comment 12 2012-10-02 13:09:07 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.