Closed Bug 799152 Opened 12 years ago Closed 12 years ago

nsExpandedPrincipal should use default extendedOrigin/appId/appStatus/isInBrowserElement

Categories

(Core :: Security: CAPS, defect)

Product:
Core
Component:
Security: CAPS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla18
Tracking Flags:
Tracking Status
firefox18 --- fixed

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(1 file)

Patch
(deleted), patch
bholley
: review+
Details | Diff | Splinter Review
Attached patch Patch (deleted) — — Splinter 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: 12 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
status-firefox18: --- → fixed
Target Milestone: mozilla19 → mozilla18
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: