IndexedDB: remove nsIVariant from IDB* interfaces

RESOLVED FIXED in mozilla10

Status

()

Core
DOM: IndexedDB
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: sicking, Assigned: Ben Turner (not reading bugmail, use the needinfo flag!))

Tracking

unspecified
mozilla10
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound])

Attachments

(1 attachment)

165.71 KB, patch
Ben Turner (not reading bugmail, use the needinfo flag!)
: review+
Details | Diff | Splinter Review
Comment hidden (empty)
Actually, this doesn't seem like a spec issue, so unblocking bug 687337.
No longer blocks: 687337
Depends on: 694145
Created attachment 571026 [details] [diff] [review]
Patch

This already has sicking's r+ in bug 694145.
Assignee: nobody → bent.mozilla
Status: NEW → ASSIGNED
Attachment #571026 - Flags: review+
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
OS: Mac OS X → All
Hardware: x86 → All
https://hg.mozilla.org/integration/mozilla-inbound/rev/ad1f423c7755
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/ad1f423c7755
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10

Updated

6 years ago
Depends on: 709977

Comment 6

5 years ago
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.
Component: DOM → DOM: IndexedDB
Target Milestone: mozilla10 → ---
Version: Trunk → unspecified
Target Milestone: --- → mozilla10
Depends on: 738985
You need to log in before you can comment on or make changes to this bug.