Closed Bug 1629689 Opened 2 years ago Closed 2 years ago

Storage.Sync Storage API Compliance

Categories

(WebExtensions :: Storage, enhancement, P2)

enhancement

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

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
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.