Closed Bug 1538262 Opened 6 years ago Closed 6 years ago

Can we remove the "ssm" usage in PrincipalVerifier::VerifyOnMainThread?

Categories

(Core :: DOM: Security, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: bzbarsky, Assigned: ckerschb)

References

Details

(Keywords: good-first-bug, Whiteboard: [domsecurity-active])

Attachments

(1 file)

After bug 1517483 the "ssm" variable is unused apart from the null check and the "error, in shutdown" return if that fails. Is that guard actually needed if we're not using "ssm" for anything else?

Keywords: good-first-bug
Priority: -- → P3
Whiteboard: [domsecurity-backlog]
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Priority: P3 → P2
Whiteboard: [domsecurity-backlog] → [domsecurity-active]
Attachment #9053666 - Attachment description: Summary: Bug 1538262: Remove unused ssm variable in PrincipalVerifier::VerifyOnMainThread. r=jkt → Bug 1538262: Remove unused ssm variable in PrincipalVerifier::VerifyOnMainThread. r=jkt
Pushed by mozilla@christophkerschbaumer.com:
https://hg.mozilla.org/integration/autoland/rev/7717e4dccf41
Remove unused ssm variable in PrincipalVerifier::VerifyOnMainThread. r=jkt
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: