Closed Bug 1546521 Opened 1 year ago Closed 1 year ago

static bool sNewCertErrorPageEnabled declared in AboutRedirector.h is unused/unreferenced

Categories

(Firefox :: Security, defect, P3)

defect

Tracking

()

RESOLVED FIXED
Firefox 68
Tracking Status
firefox68 --- fixed

People

(Reporter: jaws, Assigned: marian.meireles, Mentored)

References

(Regression)

Details

(Keywords: good-first-bug)

Attachments

(1 file)

Ah, good find, thank you. That doesn't look harmful so let's turn it into a good first bug...

Mentor: jhofmann
Flags: needinfo?(jhofmann)
Keywords: good-first-bug
Priority: -- → P3

Hey Johann, what should I do in this one?
Check if the variable is really unesed and then erase it?
Thanks!

Flags: needinfo?(jhofmann)

(In reply to Mariana Meireles from comment #2)

Hey Johann, what should I do in this one?
Check if the variable is really unesed and then erase it?
Thanks!

The variable is unused so it can safely be deleted.

Flags: needinfo?(jhofmann)

Thanks for the answer Jared.
Here's the patch: https://phabricator.services.mozilla.com/D28820

Flags: needinfo?(jaws)
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/00fcbf4c4aac
Removes unused variable sNewCertErrorPageEnabled r=jaws
Assignee: nobody → marian.meireles
Status: NEW → ASSIGNED
Flags: needinfo?(jaws)
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
You need to log in before you can comment on or make changes to this bug.