Closed Bug 1532794 Opened 6 years ago Closed 6 years ago

Investigate to remove get/set docshell.internalReferrer from propertyBag

Categories

(Core :: DOM: Security, enhancement)

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

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1532318

People

(Reporter: tnguyen, Assigned: tnguyen)

References

(Blocks 1 open bug)

Details

(Whiteboard: [domsecurity-active])

Assignee: nobody → tnguyen
Whiteboard: [domsecurity-active]

Hi Dan, I found in the above line, we have to get/set referrer from propertyBag in case referrer is empty for security reasons (for example https->http)
Now we have referrerInfo that contains originalReferrer and we could use the originalReferrer. I guess we could remove get/set referrer from propertyBag but I am not quite sure. I checked git blame for that line and I found that line was added for a long time ago for XPI whitelist. Do you know whether the XPI whitelist is still applied here or not then I could decide to remove the line?
Thanks

Flags: needinfo?(dveditz)

The old bug fixed bug 265135

I decided to remove this because I don't see xpinstall whitelisting use referrer.

Flags: needinfo?(dveditz)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.