Remove document.persist from XULDocument

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P3
normal
RESOLVED FIXED
9 months ago
a month ago

People

(Reporter: bgrins, Assigned: bgrins)

Tracking

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

9 months ago
Once Bug 1476030 lands, we aren't going to be using it anymore (instead using Services.xulStore.persist).

Comment 1

9 months ago
Thanks for the heads up, I filed bug 1476760 for TB. We've got time until this bug here lands, right?
(Assignee)

Comment 2

9 months ago
(In reply to Jorg K (GMT+2) from comment #1)
> Thanks for the heads up, I filed bug 1476760 for TB. We've got time until
> this bug here lands, right?

Yes - I'd like to get a patch up relatively shortly to prevent accidentally re-introducing callers, but there'll be a gap between Bug 1476030 landing and this.
Priority: -- → P3
Comment hidden (mozreview-request)
(Assignee)

Updated

9 months ago
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED

Comment 4

9 months ago
mozreview-review
Comment on attachment 8993956 [details]
Bug 1476678 - Remove document.persist from XULDocument;

https://reviewboard.mozilla.org/r/258564/#review265948
Attachment #8993956 - Flags: review?(bzbarsky) → review+

Comment 5

9 months ago
Pushed by bgrinstead@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cf969e57bc7b
Remove document.persist from XULDocument;r=bz

Comment 6

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/cf969e57bc7b
Status: ASSIGNED → RESOLVED
Last Resolved: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Component: DOM → DOM: Core & HTML
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.