Bug 63117 - IndexedDB createObjectStore should NOT throw if name is null
Summary: IndexedDB createObjectStore should NOT throw if name is null
Status: RESOLVED DUPLICATE of bug 63140
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-21 21:01 PDT by Mark Pilgrim (Google)
Modified: 2011-06-27 02:13 PDT (History)
6 users (show)

See Also:


Attachments
Patch (3.39 KB, patch)
2011-06-21 21:08 PDT, Mark Pilgrim (Google)
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-02 (1.44 MB, application/zip)
2011-06-21 23:22 PDT, WebKit Review Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Pilgrim (Google) 2011-06-21 21:01:35 PDT
Calling db.createObjectStore(null) should behave identically to calling db.createObjectStore("null"). I have double-checked this (highly unintuitive) behavior with the author of the WebIDL spec and he confirms it ( http://krijnhoetmer.nl/irc-logs/whatwg/20110622#l-187 ). There was also apparently a WebIDL spec bug recently asking to change this behavior, but it was WONTFIXED ( http://www.w3.org/Bugs/Public/show_bug.cgi?id=12798 ).

End result: null values for DOMString arguments get stringified to the four-character string "null". This affects several functions in WebKit's IndexedDB implementation. I am filing separate bugs for each of them. (See bug 63110 and bug 63114, for instance.)

Patch attached.
Comment 1 Mark Pilgrim (Google) 2011-06-21 21:08:14 PDT
Created attachment 98113 [details]
Patch
Comment 2 WebKit Review Bot 2011-06-21 23:22:11 PDT
Comment on attachment 98113 [details]
Patch

Attachment 98113 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/8920225
Comment 3 WebKit Review Bot 2011-06-21 23:22:16 PDT
Created attachment 98122 [details]
Archive of layout-test-results from ec2-cr-linux-02

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-02  Port: Chromium  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 4 Mark Pilgrim (Google) 2011-06-22 08:13:58 PDT
Consolidating several bugs.

*** This bug has been marked as a duplicate of bug 63140 ***
Comment 5 Hajime Morrita 2011-06-27 02:13:05 PDT
Comment on attachment 98113 [details]
Patch

Marking obsolete as the bug closed.