Summary: | IndexedDB: setVersion() version argument is required | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Pilgrim (Google) <pilgrim> | ||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | dglazkov, dgrogan, hans, tony, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Mark Pilgrim (Google)
2011-06-09 13:49:23 PDT
Created attachment 96771 [details]
Patch
Turns out we weren't checking for null either. Added a test for that too. The patch fixes both issues (missing and null). Comment on attachment 96771 [details]
Patch
LGTM
Comment on attachment 96771 [details] Patch Clearing flags on attachment: 96771 Committed r88565: <http://trac.webkit.org/changeset/88565> All reviewed patches have been landed. Closing bug. Reverted r88565 for reason: Broke Chromium browser test: IndexedDBBrowserTestWithGCExposed.DatabaseCallbacksTest Committed r88602: <http://trac.webkit.org/changeset/88602> Urk. Turns out I can't tell with certitude that this patch actually broke that particular test :( Looking at this: http://build.chromium.org/p/chromium.webkit/waterfall?builder=Linux%20Tests The test started failing with this commit, but still fails after I reverted it. Whuh? dglazkov narrowed down the error to bug 62398 (r88569). I think we need to roll this patch back in, right? (In reply to comment #8) > dglazkov narrowed down the error to bug 62398 (r88569). I think we need to roll this patch back in, right? Right. Apologies :) Comment on attachment 96771 [details]
Patch
Let's see if the commit bot can reland it for us.
Comment on attachment 96771 [details] Patch Clearing flags on attachment: 96771 Committed r88660: <http://trac.webkit.org/changeset/88660> All reviewed patches have been landed. Closing bug. (In reply to comment #12) > All reviewed patches have been landed. Closing bug. <3. |