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
Omit document.referrer for third-party requests while in ephemeral mode #3789
Conversation
EWS run on previous version of this PR (hash da14e1c) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good; you will have to add the Radar link to the commit message before merging.
da14e1c
to
2f6cd02
Compare
EWS run on current version of this PR (hash 2f6cd02) |
https://bugs.webkit.org/show_bug.cgi?id=244518 <rdar://99578273> For third-party requests, omit document.referrer when accessed from JavaScript bindings while in ephemeral mode. Reviewed by Darin Adler. * LayoutTests/http/tests/resourceLoadStatistics/omit-document-referrer-nested-third-party-iframe-ephemeral-expected.txt: Added. * LayoutTests/http/tests/resourceLoadStatistics/omit-document-referrer-nested-third-party-iframe-ephemeral.html: Added. * LayoutTests/http/tests/resourceLoadStatistics/omit-document-referrer-third-party-iframe-ephemeral-expected.txt: Added. * LayoutTests/http/tests/resourceLoadStatistics/omit-document-referrer-third-party-iframe-ephemeral.html: Added. * LayoutTests/http/tests/resourceLoadStatistics/omit-referrer-for-navigation-ephemeral-expected.txt: Added. * LayoutTests/http/tests/resourceLoadStatistics/omit-referrer-for-navigation-ephemeral.html: Added. * Source/WebCore/dom/Document+HTML.idl: * Source/WebCore/dom/Document.cpp: (WebCore::Document::referrerForBindings): * Source/WebCore/dom/Document.h: Canonical link: https://commits.webkit.org/255649@main
2f6cd02
to
09d247d
Compare
Committed 255649@main (09d247d): https://commits.webkit.org/255649@main Reviewed commits have been landed. Closing PR #3789 and removing active labels. |
09d247d
2f6cd02
π π§ͺ winπ§ͺ ios-wk2π§ͺ api-iosπ§ͺ api-macπ§ͺ api-gtkπ§ͺ mac-wk1π§ͺ mac-wk2π§ͺ mac-AS-debug-wk2