Bug 58466 – IndexedDB createIndex should fail if name arg is empty string

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

[Mark Pilgrim (Google)]

Similiar to bug 58365, but I'm creating a separate bug report because I'm not entirely sure that this behavior is a bug. (On the other hand, I'm sure bug 58365 is a bug.) But I can't find exact spec text to support this bug. Mozilla throws in this case, WebKit does not, so someone is wrong. Opening a bug in case it's us.

[Mark Pilgrim (Google)]

Created attachment 89421 [details]
test case

[Mark Pilgrim (Google)]

Created attachment 90942 [details]
patch + layouttest

Spec says several things can be an empty string, but an index name is not one of them. This is a review-ready patch that throws when trying to call createIndex with a name parameter that is an empty string. Plus a layouttest.

[David Grogan]

(In reply to comment #2)
> Created an attachment (id=90942) [details]
> patch + layouttest
> 
> Spec says several things can be an empty string, but an index name is not one of them.

Hrm, it also doesn't forbid it (that I found).  I might ask mozilla why they made it throw.

> This is a review-ready patch that throws when trying to call createIndex with a name parameter that is an empty string. Plus a layouttest.

[Mark Pilgrim (Google)]

Discussion with SickingJ leads to the conclusion that the spec will be changed to explicitly ALLOW name to be an empty string. Closing this bug.

http://lists.w3.org/Archives/Public/public-webapps/2011AprJun/0349.html