Bug 89505

Summary:

IndexedDB: implement upgradeneeded

Product:

WebKit

Reporter:

David Grogan <dgrogan>

Component:

New Bugs

Assignee:

David Grogan <dgrogan>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

abarth, levin+threading, ojan, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

90411

Attachments:

Description	Flags
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none

David Grogan

Reported 2012-06-19 13:28:02 PDT

IndexedDB: implement upgradeneeded

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
Show Obsolete (8) View All Add attachment proposed patch, testcase, etc.

David Grogan

Comment 1 2012-06-19 13:31:03 PDT

Created attachment 148406 [details] Patch

David Grogan

Comment 2 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.

David Grogan

Comment 3 2012-07-11 16:34:58 PDT

Created attachment 151813 [details] Patch

David Grogan

Comment 4 2012-07-20 20:18:19 PDT

Created attachment 153643 [details] Patch

David Grogan

Comment 5 2012-07-20 20:19:46 PDT

The master chromium patch is http://codereview.chromium.org/10803062/

David Grogan

Comment 6 2012-07-20 23:26:53 PDT

Created attachment 153651 [details] Patch