1525763 - Use browser:purge-sessionStorage instead of extension:purge-sessionStorage in SiteDataManager.jsm

Status: VERIFIED FIXED

(Toolkit :: Data Sanitization, defect, P1)

Comment 1

[Johann Hofmann [:johannh]]

Attachment: Bug 1525763 - Use browser:purge-sessionStorage instead of extension:purge-sessionStorage in SiteDataManager.jsm. r=baku,asuth

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c948bb3bd63b
Use browser:purge-sessionStorage instead of extension:purge-sessionStorage in SiteDataManager.jsm. r=asuth

Comment 3

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/c948bb3bd63b

Comment 4

[Johann Hofmann [:johannh]]

Comment on attachment 9041961 [details]
Bug 1525763 - Use browser:purge-sessionStorage instead of extension:purge-sessionStorage in SiteDataManager.jsm. r=baku,asuth

Beta/Release Uplift Approval Request

Feature/Bug causing the regression

Bug 1483440

User impact if declined

Clearing site data in the site data manager doesn't clear sessionStorage

Is this code covered by automated tests?

No

Has the fix been verified in Nightly?

No

Needs manual test from QE?

Yes

If yes, steps to reproduce

• Visit a site with sessionStorage (like https://edition.cnn.com/)
• View sessionStorage in DevTools (keep it open)
• Open the Site Data Manager in about:preferences and delete storage for edition.cnn.com
• Refresh the sessionStorage in devtools using the little refresh arrow icon
• Most sessionStorage should disappear (there may be storage left over that is being immediately reset by the page, like in my case "beemray_id4")

List of other uplifts needed

None

Risk to taking this patch

Low

Why is the change risky/not risky? (and alternatives if risky)

Just a typo fix

String changes made/needed

None

Comment 5

[Cristian Baica [:cbaica], Release Desktop QA]

I have managed to reproduce the issue using Fx67.0a1 buildID: 20190206215551.

The issue is verified fixed using Fx67.0a1 buildID: 20190213213409 on Ubuntu 16.04, macOS 10.12 and Windows 10. Most of the session storage is cleared, only 'beemray_id4' remains but is reset by the page as mentioned in c#4.

Comment 6

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 9041961 [details]
Bug 1525763 - Use browser:purge-sessionStorage instead of extension:purge-sessionStorage in SiteDataManager.jsm. r=baku,asuth

Typo fix for session storage, verified in Nightly.
OK for uplift for beta 8.

Comment 7

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/releases/mozilla-beta/rev/191b3e80ff00

Comment 8

[Cristian Baica [:cbaica], Release Desktop QA]

The issue is also verified fixed using Fx66.0b8 on Ubuntu 18.04, Windows 10 and macOS 10.14.