Remove gBrowser.serializationHelper getter

RESOLVED FIXED in Firefox 65

Status

()

enhancement
P3
normal
RESOLVED FIXED
6 months ago
5 months ago

People

(Reporter: dao, Assigned: r2hkri, Mentored)

Tracking

({good-first-bug})

unspecified
Firefox 65
Points:
---

Firefox Tracking Flags

(firefox65 fixed)

Details

(Whiteboard: [lang=js])

Attachments

(1 attachment)

(Assignee)

Comment 1

6 months ago
Hello, I would like to try to fix this. 

Is it as simple as removing the aforementioned lines? And how would testing work?

I have already built Firefox (non artifact version on Linux) so my guess is that I would just try `./mach run` and see if everything still works.

Thanks!
(Reporter)

Comment 3

5 months ago
(In reply to r2hkri from comment #1)
> Hello, I would like to try to fix this. 
> 
> Is it as simple as removing the aforementioned lines?

Yep.

> And how would testing work?
> 
> I have already built Firefox (non artifact version on Linux) so my guess is
> that I would just try `./mach run` and see if everything still works.

Yeah, that's good enough in this case.

Thanks!
Assignee: nobody → r2hkri

Comment 4

5 months ago
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/de9cc5033d87
Remove gBrowser.serializationHelper getter because it's unused r=dao

Comment 5

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/de9cc5033d87
Status: NEW → RESOLVED
Last Resolved: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
You need to log in before you can comment on or make changes to this bug.