Closed Bug 1570644 Opened 5 years ago Closed 5 years ago

Disable LSNG in 69


(Core :: Storage: localStorage & sessionStorage, task, P1)




Tracking Status
firefox69 blocking fixed
firefox70 --- unaffected


(Reporter: hsinyi, Assigned: janv)




(Keywords: regression, Whiteboard: [rca - Design Error])


(4 files)

We're working on a fix due to bug 1560697 and we've decided to disable LSNG in 69.

Assignee: nobody → jvarga
Priority: -- → P1
See Also: → 1562942
See Also: → 1560697

Comment on attachment 9083285 [details]
Bug 1570644 - Part 4: Disable LSNG in 69; r=asuth

Beta/Release Uplift Approval Request

  • User impact if declined: If LSNG is enabled, quota manager storage initialization time is increased by ~30%
    We don't want to ship LSNG until this regression is fixed.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This patch just disables LSNG, so the old implementation of LocalStorage will be used in FF 69. We fully support downgrading of LocalStorage data, so if some users already tried a beta build, they should be safe. We actually disabled LSNG in previous beta cycles and there were no problems reported after the switch.
  • String changes made/needed: None
Attachment #9083285 - Flags: approval-mozilla-beta?

Comment on attachment 9083285 [details]
Bug 1570644 - Part 4: Disable LSNG in 69; r=asuth

Uplift approved for 69 beta 12, thanks.

Attachment #9083285 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69

Backed out for various localstorage related failures, e.g. in dom/tests/browser/browser_localStorage_e10s.js:

Push with failures:
Please check all failed jobs.
Failure log for dom/tests/browser/browser_localStorage_e10s.js:
TEST-UNEXPECTED-FAIL | dom/tests/browser/browser_localStorage_e10s.js | Uncaught exception - at resource://specialpowers/SpecialPowersAPIParent.jsm:22 - SpecialPowersError: Unrecognized Special Powers API: Wakeup

Flags: needinfo?(jvarga)
Resolution: FIXED → ---

Marking this uplift as a blocker as we don't want to ship without it in 69.

Severity: normal → major

I'm verifying a patch here:

There's still some work and this will require a new review, should be ready soon.

Flags: needinfo?(jvarga)
Regressed by: 1556812
Regressed by: 1541557
Regressed by: 1551055
Keywords: regression
Attachment #9083285 - Attachment description: Bug 1570644 - Disable LSNG in 69; r=asuth → Bug 1570644 - Part 4: Disable LSNG in 69; r=asuth

Part 1, 2, 3 need to get r+ and land on m-c first.

Pushed by
Part 1: Fix a regression caused by bug 1556812; r=asuth
Part 2: Fix a regression caused by bug 1541557 part 7; r=asuth
Part 3: Fix a regression caused by bug 1551055 part 1; r=asuth
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Target Milestone: mozilla69 → mozilla70

To be clear, only the support patches landed on m-c for Fx70. The actual disabling of LSNG only landed on Beta. Updating the flags to better reflect that.

Target Milestone: mozilla70 → mozilla69

This bug has been identified as part of a pilot on determining root causes of blocking and dot release drivers.

It needs a root-cause set for it. Please see the list at

Add the root cause as a whiteboard tag in the form [rca - <cause> ] and remove the rca-needed keyword.

If you have questions, please contact :tmaity.

Keywords: rca-needed

It is difficult to point to a single root cause here. It is a mix of heritage design decisions and premature landing of new features on top of them.

Keywords: rca-needed
Whiteboard: [rca - Design Error]
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.