The following code should throw an exception on the second openDatabase() call, but doesn't: var n = 'test ' + new Date(); var db1 = openDatabase(n, '', '', 0); var db2 = openDatabase(n, 'test', '', 0); STEPS TO REPRODUCE 1. Open http://software.hixie.ch/utilities/js/js-eval-window/ 2. Paste the code above 3. Click the button EXPECTED RESULTS It should say "Error: INVALID_STATE_ERR: DOM Exception 11". ACTUAL RESULTS It says "undefined". It works fine if the first openDatabase() call's second argument is not the empty string. This may be due to some confusion about the empty string argument to openDatabase() -- per spec, the empty string sets the version to the empty string if the database doesn't exist, but otherwise just accepts whatever version is there.
I have a fix and layout test for this bug, will send the patch shortly. I've run the layout tests on Windows and Mac (debug) and with the patch all the old tests continue to pass.
Created attachment 41112 [details] Proposed patch
ping...
fwiw, lgtm
(In reply to comment #4) > fwiw, lgtm Thanks Michael. cc'ing ddkilzer who reviewed the last DB patch I sent. David, would you mind looking at this? Thanks!
The code looks fine, but I don't know which is the proper behavior.
(In reply to comment #6) > The code looks fine, but I don't know which is the proper behavior. Considering that the bug was filed by the author of HTML5, I think the correct behavior is as stated in Comment #0.
Comment on attachment 41112 [details] Proposed patch > Index: LayoutTests/storage/open-database-set-empty-version.html > =================================================================== > --- LayoutTests/storage/open-database-set-empty-version.html (revision 0) > +++ LayoutTests/storage/open-database-set-empty-version.html (revision 0) > @@ -0,0 +1,26 @@ > +<html> > +<head> > +<script> > +function runTest() { > + if (window.layoutTestController) { > + layoutTestController.dumpAsText(); > + layoutTestController.clearAllDatabases(); > + } > + > + try { > + var db = openDatabase('28417Test', '', 'Test for bug 28417: openDatabase() with empty version sets db version up incorrectly', 0); > + // The next openDatabase call should fail because the database version was set to '' by the call above, and now we are expecting a different version. > + var db2 = openDatabase('28417Test', 'test', 'Test for bug 28417: openDatabase() with empty version sets db version up incorrectly', 0); > + } catch (e) { > + document.getElementById('result').innerHTML = 'SUCCESS, an exception was thrown. ' + e; > + } The "var db = ..." statement could be moved outside the try{} block in case it generates an exception (although I suppose it's somewhat unlikely that it would generate the same exception and make the test pass when it shouldn't). Might be nice to close the databases when you're done with them, too, but not necessary. r=me
Comment on attachment 41112 [details] Proposed patch Clearing flags on attachment: 41112 Committed r50350: <http://trac.webkit.org/changeset/50350>
All reviewed patches have been landed. Closing bug.