IDBKeyRange.bound() should not use the optional options object
Created attachment 84135 [details] Patch
This moves us back in line with the spec.
Created attachment 84278 [details] Patch
Unofficial LGTM. This is indeed what the spec says and I checked for absent-minded-type mistakes we all make. Just look over for style.
Comment on attachment 84278 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=84278&action=review OK > Source/WebCore/ChangeLog:8 > + https://bugs.webkit.org/show_bug.cgi?id=55419 > + > + * storage/IDBKeyRange.cpp: Can you add a reference to the relevant tests? Link to spec text would be nice as well.
(In reply to comment #5) > (From update of attachment 84278 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=84278&action=review > > OK > > > Source/WebCore/ChangeLog:8 > > + https://bugs.webkit.org/show_bug.cgi?id=55419 > > + > > + * storage/IDBKeyRange.cpp: > > Can you add a reference to the relevant tests? Link to spec text would be nice as well. Will add http://dvcs.w3.org/hg/IndexedDB/raw-file/tip/Overview.html#range-concept and a note about the existing tests that were updated. Thanks.
Committed r80076: <http://trac.webkit.org/changeset/80076>