Closed Bug 391438 Opened 17 years ago Closed 17 years ago

[FIXr]Remove originURI argument from contentpolicy calls

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: sicking, Assigned: bzbarsky)

Details

Attachments

(1 file)

This is a spinoff from bug 388597 comment 19. We can remove the originURI argument from NS_CheckContent(Load|Process)Policy if we in nsDocShell::InternalLoad call GetCodebasePrincipal/GetSystemPrincipal for the passed in uri.
Attached patch Like so?Splinter Review
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #275915 - Flags: superreview?(jonas)
Attachment #275915 - Flags: review?(jonas)
OS: Windows XP → All
Hardware: PC → All
Summary: Remove originURI argument from contentpolicy calls → [FIX]Remove originURI argument from contentpolicy calls
Attachment #275915 - Flags: superreview?(jonas)
Attachment #275915 - Flags: superreview+
Attachment #275915 - Flags: review?(jonas)
Attachment #275915 - Flags: review+
Comment on attachment 275915 [details] [diff] [review]
Like so?

Requesting 1.9 approval.  Safe fix, no behavior change.  Just slightly cleaner setup for future work.
Attachment #275915 - Flags: approval1.9?
Summary: [FIX]Remove originURI argument from contentpolicy calls → [FIXr]Remove originURI argument from contentpolicy calls
Attachment #275915 - Flags: approval1.9? → approval1.9+
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: