FUEL: cleanup for SessionStorage

RESOLVED FIXED

Status

()

Firefox
General
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: mfinkle, Assigned: mfinkle)

Tracking

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 271256 [details] [diff] [review]
switch to nsIVariant instead of AString for storage data

SessionStorage currently holds strings. This should be changed to allow any JS type. Switching to nsIVariant should work and cause no compatibility problems with existing code.
Attachment #271256 - Flags: review?(gavin.sharp)
(Assignee)

Updated

11 years ago
Status: NEW → ASSIGNED

Updated

11 years ago
Assignee: nobody → mark.finkle
Status: ASSIGNED → NEW
Comment on attachment 271256 [details] [diff] [review]
switch to nsIVariant instead of AString for storage data

Bump the IID, too? I know it makes effectively no difference in this case, but it makes me feel warm and fuzzy ;)
Attachment #271256 - Flags: review?(gavin.sharp) → review+
(Assignee)

Comment 2

11 years ago
Bumped IID and landed
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.