Closed Bug 1596542 Opened 1 year ago Closed 11 months ago

Only run browsingData API "test_browserData_should_not_remove_extension_data" test case if LSNG is enabled

Categories

(WebExtensions :: Storage, task)

task
Not set
normal

Tracking

(firefox72 fixed)

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: rpl, Assigned: rpl)

Details

Attachments

(1 file)

Bug 1575891 uplift to beta has been backed out because of a failure triggered by this test when LSNG is not enabled.

To avoid the test to fail again if we have to disable LSNG also in Firefox 72 (which should be unlikely, but still better if we can make it simpler in case we have to) we should at least skip this particular test case when LSNG is turned off explicitly.

When LSNG is disabled, the browsingData API does clear the localStorage data by notify a "extension:purge-localStorage"
message over the observer service, when the browserData API call doesn't specify a list of specific origins to clear,
the code that handles "extension:purge-localStorage" will clear all the localStorage data, including the one stored by extensions:

Assignee: nobody → lgreco
Status: NEW → ASSIGNED
Pushed by luca.greco@alcacoop.it:
https://hg.mozilla.org/integration/autoland/rev/d79432359063
Skip test_browserData_should_not_remove_extension_data if LSNG is disabled. r=mixedpuppy
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72

Set qe-verify-, as this is a test-only change.

Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.