1572191 - Don't persist nsIPermissionManager db to disk in GV

Status: RESOLVED FIXED

(GeckoView :: General, defect, P2)

Description

[Dylan Roeh (:droeh) (he/him)]

We don't want the permission manager to persist anything to disk in GV; anything necessary should be handled by the app.

Comment 1

[Dylan Roeh (:droeh) (he/him)]

Attachment: Bug 1572191 - Wipe permissions db on startup in GeckoView. r=snorp!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/987353e173e1
Set memory-only permission manager DB pref in GV. r=snorp

Comment 3

[Bogdan Tara[:bogdan_tara | bogdant]]

Backed out changeset 987353e173e1 (Bug 1572191) for triggering many android failures

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&searchStr=Android&tochange=ca86d788d9bb7b77ec64cbc3e58124effc58d22f&fromchange=e7da23f768b5b9668c6b337cb5c6ba50530bbe9a&selectedJob=260608276

Backout link: https://hg.mozilla.org/integration/autoland/rev/ca86d788d9bb7b77ec64cbc3e58124effc58d22f

Failures log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=260608287&repo=autoland&lineNumber=3093

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=260608267&repo=autoland&lineNumber=1972

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=260608276&repo=autoland&lineNumber=23005

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=260608296&repo=autoland&lineNumber=2205

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=260610951&repo=autoland&lineNumber=1963

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=260610922&repo=autoland&lineNumber=23010

Comment 4

[Chris Peterson [:cpeterson]]

Dylan says the permission manager includes other non-site-specific data that we do want to preserve. He will need to selectively choose which data to save.

Comment 5

[Dylan Roeh (:droeh) (he/him)]

Attachment: Bug 1572191 - Prevent nsPermissionManager from persisting certain permissions to disk under GeckoView. r=snorp!,ehsan!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a08f7d112fbf
Prevent nsPermissionManager from persisting certain permissions to disk under GeckoView. r=snorp,Ehsan

Comment 7

[Dorel Luca [:dluca]]

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

Comment 8

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/releases/mozilla-beta/rev/a08f7d112fbf