Closed Bug 1369835 Opened 7 years ago Closed 7 years ago

Port bug 1369323 |Let's remove nsScriptSecurityManager::GetNoAppCodebasePrincipal| to TB

Categories

(Thunderbird :: General, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 55.0

People

(Reporter: jorgk-bmo, Assigned: jorgk-bmo)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1369323 +++
This method is just used by some ffos related addons.

===

Well, unfortunately this method is used in TB:
https://dxr.mozilla.org/comm-central/search?q=getNoAppCodebasePrincipal&redirect=false

and removing it now causes errors seen on treeherder here:
https://archive.mozilla.org/pub/thunderbird/tinderbox-builds/comm-central-macosx64/1496329232/comm-central_yosemite_r7_test-mozmill-bm133-tests1-macosx-build22.txt.gz

JavaScript error: chrome://messenger/content/newmailaccount/accountProvisioner.js, line 42: TypeError: Services.scriptSecurityManager.getNoAppCodebasePrincipal is not a function

Let's hope that the replacement
-          principal: Services.scriptSecurityManager.getNoAppCodebasePrincipal(targetURI)
+          principal: Services.scriptSecurityManager.createCodebasePrincipal(targetURI, {}),
will just work ;-)
Assignee: nobody → jorgk
Status: NEW → ASSIGNED
https://hg.mozilla.org/comm-central/rev/22d34b082d5c3b9498d424781b3b24e8f3c536f0
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 55.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: