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)
Core
DOM: Security
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)
(deleted),
text/x-phabricator-request
|
Details |
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?
Updated•6 years ago
|
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Priority: P3 → P2
Whiteboard: [domsecurity-backlog] → [domsecurity-active]
Updated•6 years ago
|
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
Comment 3•6 years ago
|
||
bugherder |
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.
Description
•