Summary: | IndexedDB delete() should fail if key is null | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Pilgrim (Google) <pilgrim> | ||||||||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | commit-queue, dgrogan, fishd, hans, pilgrim | ||||||||||||
Priority: | P3 | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | All | ||||||||||||||
OS: | All | ||||||||||||||
Attachments: |
|
Description
Mark Pilgrim (Google)
2011-04-14 17:42:52 PDT
Created attachment 89702 [details]
test case
Created attachment 91097 [details]
patch with changelog and layouttest
patch to check if key is null in IDBObjectStoreBackendImpl.cpp::deleteFunction
Comment on attachment 91097 [details] patch with changelog and layouttest View in context: https://bugs.webkit.org/attachment.cgi?id=91097&action=review Code LG, just nits. > third_party/WebKit/Source/WebCore/ChangeLog:5 > + IndexedDB object store delete should fail if key is null make more obvious that you're referring to an object store deleting an entry, not an entire object store being deleted. e.g. myObjectStore.delete(null) > third_party/WebKit/LayoutTests/storage/indexeddb/mozilla/key-requirements-delete-null-key.html:45 > +function cleanDatabase() Change to something not misleading, e.g. cleanDatabaseAndCreateObjectStore or just inSetVersion Comment on attachment 91097 [details]
patch with changelog and layouttest
Feel free to upload a patch addressing David's nits and I will cq+ it.
Created attachment 91188 [details]
nits addressed
Comment on attachment 91188 [details]
nits addressed
LGTM
Created attachment 91212 [details]
rebase webkit directory
Created attachment 91284 [details]
nits addressed once again (regression in previous patch)
Comment on attachment 91284 [details] nits addressed once again (regression in previous patch) Clearing flags on attachment: 91284 Committed r85145: <http://trac.webkit.org/changeset/85145> All reviewed patches have been landed. Closing bug. |