692669 - IndexedDB: remove nsIVariant from IDB* interfaces

Status: RESOLVED FIXED

(Core :: Storage: IndexedDB, defect)

Comment 1

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Actually, this doesn't seem like a spec issue, so unblocking bug 687337.

Comment 2

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Attachment: Patch

This already has sicking's r+ in bug 694145.

Comment 3

[Ed Morley [:emorley]]

Original landing (https://hg.mozilla.org/integration/mozilla-inbound/rev/eb32aca6d54d) + attempt at bustage fix (https://hg.mozilla.org/integration/mozilla-inbound/rev/5675bf2c7930) backed out for failing to build on any platform:
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=5675bf2c7930

https://hg.mozilla.org/integration/mozilla-inbound/rev/72042cd154b9

Comment 4

[Ben Turner (not reading bugmail, use the needinfo flag!)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ad1f423c7755

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/ad1f423c7755

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

IDBKeyRange::FromJSVal could possibly use nsContentUtils::XPConnect()->WrapJS.

Also, I wonder if Bug 709569 could reduce some manual JS API usage, or if some new features
should be added to the tool.