1570644 - Disable LSNG in 69

Status: RESOLVED FIXED

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

Description

[Hsin-Yi Tsai (away for a while) [:hsinyi]]

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

Comment 1

[Jan Varga [:janv]]

Attachment: Bug 1570644 - Part 4: Disable LSNG in 69; r=asuth

Comment 2

[Jan Varga [:janv]]

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

Comment 3

[Pascal Chevrel:pascalc]

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

Uplift approved for 69 beta 12, thanks.

Comment 4

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/releases/mozilla-beta/rev/1caee43f3272

Comment 5

[Sebastian Hengst [:aryx] (needinfo on intermittent or backout)]

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

https://hg.mozilla.org/releases/mozilla-beta/rev/8055f364e2cc60f0f721ec2694f684b715a2c037

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&selectedJob=260447733&resultStatus=usercancel%2Ctestfailed%2Cbusted%2Cexception&revision=440c7ba936c5a5344ecd5be2801f1e8da229d361
Please check all failed jobs.
Failure log for dom/tests/browser/browser_localStorage_e10s.js: https://treeherder.mozilla.org/logviewer.html#?job_id=260447733&repo=mozilla-beta
TEST-UNEXPECTED-FAIL | dom/tests/browser/browser_localStorage_e10s.js | Uncaught exception - at resource://specialpowers/SpecialPowersAPIParent.jsm:22 - SpecialPowersError: Unrecognized Special Powers API: Wakeup

Comment 6

[Pascal Chevrel:pascalc]

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

Comment 7

[Jan Varga [:janv]]

I'm verifying a patch here:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e3d7fe0f334687c45a1143fca0d132528eb1a7b4

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

Comment 8

[Jan Varga [:janv]]

This looks better:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=aaab0a31862a1cfb3a3870bcd7ce13c497ad3f3d

Comment 9

[Jan Varga [:janv]]

Attachment: Bug 1570644 - Part 1: Fix a regression caused by bug 1556812; r=asuth

Comment 10

[Jan Varga [:janv]]

Attachment: Bug 1570644 - Part 2: Fix a regression caused by bug 1541557 part 7; r=asuth

Comment 11

[Jan Varga [:janv]]

Attachment: Bug 1570644 - Part 3: Fix a regression caused by bug 1551055 part 1; r=asuth

Comment 12

[Jan Varga [:janv]]

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

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/60cb55ea9f2a
Part 1: Fix a regression caused by bug 1556812; r=asuth
https://hg.mozilla.org/integration/autoland/rev/7aa755add6a0
Part 2: Fix a regression caused by bug 1541557 part 7; r=asuth
https://hg.mozilla.org/integration/autoland/rev/6ead2cb63757
Part 3: Fix a regression caused by bug 1551055 part 1; r=asuth

Comment 14

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/77569c67c0e0
https://hg.mozilla.org/releases/mozilla-beta/rev/6abd377f1816
https://hg.mozilla.org/releases/mozilla-beta/rev/ee83ffabc109
https://hg.mozilla.org/releases/mozilla-beta/rev/62db9e3b8d6f

Comment 15

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/60cb55ea9f2a
https://hg.mozilla.org/mozilla-central/rev/7aa755add6a0
https://hg.mozilla.org/mozilla-central/rev/6ead2cb63757

Comment 16

[Ryan VanderMeulen [:RyanVM]]

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.

Comment 17

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

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 https://docs.google.com/document/d/1FFEGsmoU8T0N8R9kk-MXWptOPtXXXRRIe4vQo3_HgMw/.

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.

Comment 18

[Jens Stutte [:jstutte]]

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.