WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
62322
IndexedDB test: odd database names
https://bugs.webkit.org/show_bug.cgi?id=62322
Summary
IndexedDB test: odd database names
Mark Pilgrim (Google)
Reported
2011-06-08 14:10:12 PDT
http://dvcs.w3.org/hg/IndexedDB/raw-file/tip/Overview.html#database-concept
states "Every database has a name which identifies it within a specific origin. The name can be any string value. ... Implementations must support all names. If a implementation uses a storage mechanism which can't handle arbitrary database names, the implementation must use an escaping mechanism or something similar to map the provided name to a name that it can handle." This test creates a series of databases with names as odd Unicode characters (null, surrogate code points, non-BMP characters, etc) that are traditionally problematic for storage systems. WebKit passes this test.
Attachments
Patch
(4.43 KB, patch)
2011-06-08 14:11 PDT
,
Mark Pilgrim (Google)
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Mark Pilgrim (Google)
Comment 1
2011-06-08 14:11:32 PDT
Created
attachment 96476
[details]
Patch
David Grogan
Comment 2
2011-06-08 14:15:48 PDT
Comment on
attachment 96476
[details]
Patch Could you add another couple of steps here: add data to each objectstore, close it, reopen it with the crazy name and ensure the data is still there?
Mark Pilgrim (Google)
Comment 3
2011-06-08 20:08:56 PDT
Good call, there are some string keys which are not stored and/or retrieved properly. Tracked separately as
bug 62348
.
Eric Seidel (no email)
Comment 4
2011-06-13 15:21:36 PDT
Comment on
attachment 96476
[details]
Patch I like tests!
WebKit Review Bot
Comment 5
2011-06-13 15:30:01 PDT
Comment on
attachment 96476
[details]
Patch Clearing flags on attachment: 96476 Committed
r88713
: <
http://trac.webkit.org/changeset/88713
>
WebKit Review Bot
Comment 6
2011-06-13 15:30:05 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug