Bug 89505 - IndexedDB: implement upgradeneeded
Summary: IndexedDB: implement upgradeneeded
Status: RESOLVED DUPLICATE of bug 92897
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Grogan
URL:
Keywords:
Depends on:
Blocks: 90411
  Show dependency treegraph
 
Reported: 2012-06-19 13:28 PDT by David Grogan
Modified: 2012-08-21 15:27 PDT (History)
4 users (show)

See Also:


Attachments
Patch (89.18 KB, patch)
2012-06-19 13:31 PDT, David Grogan
no flags Details | Formatted Diff | Diff
Patch (121.21 KB, patch)
2012-07-11 16:34 PDT, David Grogan
no flags Details | Formatted Diff | Diff
Patch (116.13 KB, patch)
2012-07-20 20:18 PDT, David Grogan
no flags Details | Formatted Diff | Diff
Patch (120.63 KB, patch)
2012-07-20 23:26 PDT, David Grogan
no flags Details | Formatted Diff | Diff
Patch (144.28 KB, patch)
2012-07-26 19:12 PDT, David Grogan
no flags Details | Formatted Diff | Diff
Patch (143.28 KB, patch)
2012-07-26 20:44 PDT, David Grogan
no flags Details | Formatted Diff | Diff
Patch (143.73 KB, patch)
2012-07-26 20:53 PDT, David Grogan
no flags Details | Formatted Diff | Diff
Patch (147.42 KB, patch)
2012-07-26 22:44 PDT, David Grogan
no flags Details | Formatted Diff | Diff
Patch (147.91 KB, patch)
2012-07-26 22:57 PDT, David Grogan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Grogan 2012-06-19 13:28:02 PDT
IndexedDB: implement upgradeneeded
Comment 1 David Grogan 2012-06-19 13:31:03 PDT
Created attachment 148406 [details]
Patch
Comment 2 David Grogan 2012-06-19 13:35:33 PDT
This isn't ready to be reviewed, I mostly wanted it to be backed up.

The status:
calling open("db", 7) fires upgradeneeded, runs the version change transaction, fires complete, and then fires success with the database handle.

calling db.version doesn't work yet, it knows nothing about the int versions.
Comment 3 David Grogan 2012-07-11 16:34:58 PDT
Created attachment 151813 [details]
Patch
Comment 4 David Grogan 2012-07-20 20:18:19 PDT
Created attachment 153643 [details]
Patch
Comment 5 David Grogan 2012-07-20 20:19:46 PDT
The master chromium patch is http://codereview.chromium.org/10803062/
Comment 6 David Grogan 2012-07-20 23:26:53 PDT
Created attachment 153651 [details]
Patch
Comment 7 David Grogan 2012-07-26 19:12:04 PDT
Created attachment 154811 [details]
Patch
Comment 8 David Grogan 2012-07-26 20:44:34 PDT
Created attachment 154824 [details]
Patch
Comment 9 David Grogan 2012-07-26 20:53:18 PDT
Created attachment 154825 [details]
Patch
Comment 10 David Grogan 2012-07-26 22:44:43 PDT
Created attachment 154844 [details]
Patch
Comment 11 David Grogan 2012-07-26 22:57:04 PDT
Created attachment 154847 [details]
Patch
Comment 12 David Grogan 2012-08-21 15:27:18 PDT

*** This bug has been marked as a duplicate of bug 92897 ***