Closed Bug 1272590 Opened 8 years ago Closed 8 years ago

Remove SEC_NORMAL from toolkit/content/contentAreaUtils.js

Categories

(Core :: DOM: Security, defect)

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

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla49
Tracking Flags:
Tracking Status
firefox49 --- fixed

People

(Reporter: ckerschb, Assigned: ckerschb)

References

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

bug_1272590_remove_sec_normal_contentareutils.patch
2.48 KB, patch
sicking
: review+
Details | Diff | Splinter Review 
      No description provided.
Assignee: nobody → ckerschb
Blocks: 1182535
Status: NEW → ASSIGNED
Whiteboard: [domsecurity-active]
Jonas, I think we shouldn't hold back on removing the SEC_NORMAL bit from this file even though the uriLoader has not been converted to open the channel using asyncOpen2(). Since we are loading using the systemPrincipal the assertions within asyncOpen() don't get triggered.
Attachment #8752099 - Flags: review?(jonas)
https://hg.mozilla.org/mozilla-central/rev/5d9313e120f3
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: