1331179 - storage.sync API error: "Please set webextensions.storage.sync.enabled to true in about:config"

Status: RESOLVED WONTFIX

(WebExtensions :: Compatibility, defect)

Description

[Rob Wu [:robwu]]

Bug 1253740 introduced the browser.storage.sync API, behind a flag.

Unfortunately, the browser.storage.sync API is exposed even when the API is disabled:

    var storage = chrome.storage.sync || chrome.storage.local;
    // Used as storage.set(...), storage.get(...), etc.

The above code assumes that chrome.storage.sync is functioning when it is defined. This appears to not be the case, and as a result the extension loses its ability to persist settings, and all storage.sync.* API calls fail with "Please set webextensions.storage.sync.enabled to true in about:config":

Can the chrome.storage.sync API be hidden unless the flag is enabled?
Or, treat chrome.storage.sync as chrome.storage.local (distinct from the actual storage of chrome.storage.local) when the browser does not sync (just like Chrome).

Comment 1

[Andy McKay]

Given that we are about to flip this pref on and turn on chrome.storage.sync by default, in bug 1331467, I don't think we need to worry about this.