Closed Bug 1476678 Opened 2 years ago Closed 2 years ago

Remove document.persist from XULDocument

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: bgrins, Assigned: bgrins)

References

Details

Attachments

(1 file)

Once Bug 1476030 lands, we aren't going to be using it anymore (instead using Services.xulStore.persist).
Thanks for the heads up, I filed bug 1476760 for TB. We've got time until this bug here lands, right?
(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
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
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+
Pushed by bgrinstead@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cf969e57bc7b
Remove document.persist from XULDocument;r=bz
https://hg.mozilla.org/mozilla-central/rev/cf969e57bc7b
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.