Closed
Bug 1528697
Opened 6 years ago
Closed 5 years ago
Refactor and move ReferrerPolicy.h from netwerk to dom security
Categories
(Core :: DOM: Security, enhancement)
Core
DOM: Security
Tracking
()
RESOLVED
FIXED
mozilla70
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: tnguyen, Assigned: tnguyen)
References
(Blocks 1 open bug)
Details
(Whiteboard: [domsecurity-active])
Attachments
(3 files)
There's a couple of things I would like to do in this bug:
- Move ReferrerPolicy enum https://searchfox.org/mozilla-central/rev/ee40541496d3ad738097eebadaf4965ca1343b7a/dom/webidl/Request.webidl#78
to seperate webidl file. - Expose the value generated from webidl directly https://searchfox.org/mozilla-central/source/__GENERATED__/dist/include/mozilla/dom/RequestBinding.h#137 and remove the value in nsIHttpChannel.idl
- Refactor ReferrerPolicy.h
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•5 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•5 years ago
|
||
ReferrerPolicy gets tossed back and forth as a uint32_t and
ReferrerPolicy enum in header file. Expose ReferrerPolicyValues from
webidl file and use consistently in native code.
Assignee | ||
Comment 3•5 years ago
|
||
Depends on D41954
Assignee | ||
Comment 4•5 years ago
|
||
Depends on D41955
Pushed by tnguyen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fccf3343d88d
Expose ReferrerPolicy.webidl and use referrerpolicy enum r=smaug
https://hg.mozilla.org/integration/autoland/rev/4a00b6949b8e
Add cenum referrerpolicy in ReferrerInfo.idl r=ckerschb,Gijs
https://hg.mozilla.org/integration/autoland/rev/a642029c8e7e
Remove ReferrerPolicy.h and add utils to ReferrerInfo r=ckerschb
Comment 6•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/fccf3343d88d
https://hg.mozilla.org/mozilla-central/rev/4a00b6949b8e
https://hg.mozilla.org/mozilla-central/rev/a642029c8e7e
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in
before you can comment on or make changes to this bug.
Description
•