Bug 63140 – IndexedDB: fix multiple issues in IDBDatabase.idl

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Mark Pilgrim (Google)]

This is a consolidated patch that fixes the following issues, some of which were previously reported in separate and overlapping bugs:

- createObjectStore() should throw TypeError if required name argument is missing (bug 62414)
- createObjectStore() should treat null name argument as "null" (bug 63117)
- deleteObjectStore() should throw TypeError if required name argument is missing (bug 62415)
- deleteObjectStore() should treat null name argument as "null"
- setVersion() should throw TypeError if required version argument is missing (bug 62401)
- setVersion() should treat null version argument as "null" (bug 62401)
- transaction() should throw TypeError if required storeNames argument is missing (bug 62416)
- transaction() should treat null storeNames argument as "null"

[Mark Pilgrim (Google)]

Created attachment 98174 [details]
Patch

[Mark Pilgrim (Google)]

*** Bug 62414 has been marked as a duplicate of this bug. ***

[Mark Pilgrim (Google)]

*** Bug 63117 has been marked as a duplicate of this bug. ***

[Mark Pilgrim (Google)]

*** Bug 62415 has been marked as a duplicate of this bug. ***

[Mark Pilgrim (Google)]

*** Bug 62416 has been marked as a duplicate of this bug. ***

[Adam Barth]

Comment on attachment 98174 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=98174&action=review

> Source/WebCore/storage/IDBDatabase.idl:58
>          // EventTarget interface
>          void addEventListener(in DOMString type,
>                                in EventListener listener,
> -                              in boolean useCapture);
> +                              in [Optional] boolean useCapture);
>          void removeEventListener(in DOMString type,
>                                   in EventListener listener,
> -                                 in boolean useCapture);
> +                                 in [Optional] boolean useCapture);

There's some question about what the right thing here is globally, but I'm pretty sure this is what we're going to end up with.  The only question is whether we want to make that decision all-at-once or whether we want to convert these piecemeal.  I think it's ok to convert them piecemeal.

[WebKit Review Bot]

Comment on attachment 98174 [details]
Patch

Clearing flags on attachment: 98174

Committed r89451: <http://trac.webkit.org/changeset/89451>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.