Closed
Bug 1629689
Opened 5 years ago
Closed 4 years ago
Storage.Sync Storage API Compliance
Categories
(WebExtensions :: Storage, enhancement, P2)
WebExtensions
Storage
Tracking
(firefox77 fixed)
RESOLVED
FIXED
mozilla77
Tracking | Status | |
---|---|---|
firefox77 | --- | fixed |
People
(Reporter: lougenia, Assigned: lougenia)
Details
(Whiteboard: SACI)
Attachments
(1 file)
We deal with JSON in and out - there are many variations possible. This doesn’t affect the “concrete” API, just the shape of the JS objects.
We need to ensure we comply with not only the vague spec, but also the existing implementations.
There are some tests in test_ext_storage.js, all of which pass in bug 1623245, but we probably need more (eg, there aren’t tests for “strange” inputs, such as non-string values, etc.) - but all in all, this is looking OK!
Note: This bug is a Bugzilla representation of SYNC-951.
┆Issue is synchronized with a GitHub issue
┆Issue Number: 2838
Assignee | ||
Comment 1•5 years ago
|
||
Updated•5 years ago
|
Priority: -- → P2
Pushed by mhammond@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c81fcd640993 Added additional extension storage tests for storage API compliance r=lina,mixedpuppy
Comment 3•4 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox77:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in
before you can comment on or make changes to this bug.
Description
•