Skip to content

/ servo

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement referrer policy delivery via referrer attribute #11861

Open
rebstar6 opened this issue Jun 25, 2016 · 6 comments
Open

Implement referrer policy delivery via referrer attribute #11861

rebstar6 opened this issue Jun 25, 2016 · 6 comments
Labels
A-content/dom

Comments

@rebstar6
Copy link
Contributor

@rebstar6 rebstar6 commented Jun 25, 2016

See https://w3c.github.io/webappsec-referrer-policy/#referrer-policy-delivery-referrer-attribute for details

Reference #10311 for servo referrer policy implementation thus far.
@rebstar6 rebstar6 mentioned this issue Jun 25, 2016
Open
@jdm jdm added the A-content/dom label Jun 27, 2016
@jdm
Copy link
Member

@jdm jdm commented Jun 27, 2016

This will be very similar to the implementation of #11862.
@jdm jdm mentioned this issue Jun 27, 2016
Closed
@TheKK
Copy link
Contributor

@TheKK TheKK commented Sep 21, 2016

I'd like to work on this one, Could you assign it to me :)
@jdm jdm added the C-assigned label Sep 21, 2016
@jdm
Copy link
Member

@jdm jdm commented Sep 21, 2016

Please do!
@TheKK TheKK mentioned this issue Sep 23, 2016
Merged
bors-servo added a commit to servo/string-cache that referenced this issue Sep 23, 2016
@bors-servo
Auto merge of #173 - TheKK:add_referrerPolicy, r=Manishearth
Loading status checks…
1e347fc 
Add "referrerPolicy" to static_atom_list

for servo/servo#11861

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/string-cache/173)
<!-- Reviewable:end -->
@TheKK
Copy link
Contributor

@TheKK TheKK commented Oct 5, 2016

It's kind of shame, I found that my test for HTMLAnchorElement in #12493 was totally wrong. Before sending PR, should I open issue for it first?
@jdm
Copy link
Member

@jdm jdm commented Oct 5, 2016

No need for an issue if you've got a fix ready!
@TheKK TheKK mentioned this issue Mar 25, 2017
Merged
@nox
Copy link
Member

@nox nox commented Oct 2, 2017

@TheKK Are you still working on this?
@jdm jdm removed the C-assigned label Mar 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-content/dom
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@jdm @nox @rebstar6 @TheKK
You can’t perform that action at this time.