Get rid of nsScriptSecurityManager::GetAppCodebasePrincipal

RESOLVED FIXED in Firefox 55

Status

()

Core
DOM
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: baku, Assigned: baku)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

11 months ago
This has been deprecated since 2015, and it is used only in 1 xpcshell test.
(Assignee)

Comment 1

11 months ago
Created attachment 8873341 [details] [diff] [review]
part 1 - getAppCodebasePrincipal
Attachment #8873341 - Flags: review?(bobbyholley)
(Assignee)

Updated

11 months ago
Attachment #8873341 - Attachment description: getAppCodebasePrincipal.patch → part 1 - getAppCodebasePrincipal
(Assignee)

Comment 2

11 months ago
Created attachment 8873345 [details] [diff] [review]
part 2 - fixing comments
Attachment #8873345 - Flags: review?(bobbyholley)

Updated

11 months ago
Attachment #8873341 - Flags: review?(bobbyholley) → review+

Updated

11 months ago
Attachment #8873345 - Flags: review?(bobbyholley) → review+

Comment 3

11 months ago
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f46160bbc4ec
Get rid of nsScriptSecurityManager::GetAppCodebasePrincipal, r=bholley
https://hg.mozilla.org/integration/mozilla-inbound/rev/79b91cee64f1
Fixing comments related to appId in nsIScriptSecurityManager, r=bholley

Comment 4

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f46160bbc4ec
https://hg.mozilla.org/mozilla-central/rev/79b91cee64f1
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.