Summary: | A first step towards the Indexed Database API | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jeremy Orlow <jorlow> | ||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | andreip, dglazkov, fishd, japhet | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Other | ||||||
OS: | OS X 10.5 | ||||||
Attachments: |
|
Description
Jeremy Orlow
2010-01-29 13:01:51 PST
Created attachment 47730 [details]
Patch
Comment on attachment 47730 [details] Patch One comment: > + if (args.Length() < 2) > + return throwError(V8Proxy::TypeError); Should we match http://trac.webkit.org/browser/trunk/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp#L452 and throw a SyntaxError with a similar error message? Comment on attachment 47730 [details]
Patch
What is the difference between IDBDatabaseError and SQLError? They look really, really similar. Any way we could reuse one in place of another? Ditto with IDatabaseException.
The SYNTAX_ERROR change Nate suggested is good too.
Comment on attachment 47730 [details]
Patch
I think it shouldn't stop the patch from landing though. r=me after Nate's rename.
(In reply to comment #4) > (From update of attachment 47730 [details]) > I think it shouldn't stop the patch from landing though. r=me after Nate's > rename. Rename done. Would rather combine the 2 IDLs only if the specs themselves do (maybe a common interface could be specced in the HTML spec?). Committed r54085: <http://trac.webkit.org/changeset/54085> |