Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1737)

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/resources/referrer-policy-iframe.html

Issue 2883373002: Upstream service worker "referer" tests to WPT (Closed)
Patch Set: Incorporate review feedback Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/resources/referrer-policy-iframe.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/referrer-policy-iframe.html b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/resources/referrer-policy-iframe.html
similarity index 59%
rename from third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/referrer-policy-iframe.html
rename to third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/resources/referrer-policy-iframe.html
index 82b5e23570554c95262e099bc0aaf8ccc949b900..9ef3cd19a98246989041ec08a53ba8a31aa67ecf 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/referrer-policy-iframe.html
+++ b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/resources/referrer-policy-iframe.html
@@ -1,15 +1,16 @@
-<script src="../../resources/get-host-info.js"></script>
+<script src="/common/get-host-info.sub.js"></script>
+<script src="test-helpers.sub.js"></script>
<script>
function check_referer(url, expected_referer) {
return fetch(url)
.then(function(res) { return res.json(); })
.then(function(headers) {
- if (headers['Referer'] === expected_referer) {
+ if (headers['referer'] === expected_referer) {
return Promise.resolve();
} else {
return Promise.reject('Referer for ' + url + ' must be ' +
expected_referer + ' but got ' +
- headers['Referer']);
+ headers['referer']);
}
});
}
@@ -17,13 +18,13 @@ function check_referer(url, expected_referer) {
window.addEventListener('message', function(evt) {
var host_info = get_host_info();
var port = evt.ports[0];
- check_referer('request-headers.php?ignore=true',
- host_info['HTTP_ORIGIN'] +
- '/serviceworker/resources/referrer-policy-iframe.html')
+ check_referer('request-headers.py?ignore=true',
+ host_info['HTTPS_ORIGIN'] +
+ base_path() + 'referrer-policy-iframe.html')
.then(function() {
return check_referer(
- 'request-headers.php?url=request-headers.php',
- host_info['HTTP_ORIGIN'] + '/');
+ 'request-headers.py?url=request-headers.py',
+ host_info['HTTPS_ORIGIN'] + '/');
})
.then(function() { port.postMessage({results: 'finish'}); })
.catch(function(e) { port.postMessage({results: 'failure:' + e}); });

Powered by Google App Engine
This is Rietveld 408576698