Closed Bug 799152 Opened 8 years ago Closed 8 years ago

nsExpandedPrincipal should use default extendedOrigin/appId/appStatus/isInBrowserElement

Categories

(Core :: Security: CAPS, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla18
Tracking Status
firefox18 --- fixed

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
In order to have bug 777467 working, we need to be able to call GetAppId(), GetExtendedOrigin(), ... for all principals. However, nsExpandedPrincipal do not return values for those. Seems better to have them return the same value as System and Null Principals which can be considered as default values.
Attachment #669191 - Flags: review?(bobbyholley+bmo)
Blocks: 789224
Attachment #669191 - Flags: review?(bobbyholley+bmo) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/8a20f6ef0df1
No longer blocks: 789224
Target Milestone: --- → mozilla19
https://hg.mozilla.org/mozilla-central/rev/8a20f6ef0df1
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Landed on aurora with a=blocking-basecamp given that it's blocking a basecamp blocker.
https://hg.mozilla.org/releases/mozilla-aurora/rev/668901e8d3f5
Target Milestone: mozilla19 → mozilla18
You need to log in before you can comment on or make changes to this bug.