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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/security/upgrade-insecure-requests/iframe-upgrade-csp.https.html

Issue 2068443002: Revert of Move 'frame-src' CSP checks into FrameFetchContext. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: run revert on local machine to resolve conflicts Created 4 years, 6 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/http/tests/security/upgrade-insecure-requests/iframe-upgrade-csp.https.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/upgrade-insecure-requests/iframe-upgrade-csp.https.html b/third_party/WebKit/LayoutTests/http/tests/security/upgrade-insecure-requests/iframe-upgrade-csp.https.html
deleted file mode 100644
index 1e4ed46597204a3c3015f9230f3e832e9a5add9e..0000000000000000000000000000000000000000
--- a/third_party/WebKit/LayoutTests/http/tests/security/upgrade-insecure-requests/iframe-upgrade-csp.https.html
+++ /dev/null
@@ -1,46 +0,0 @@
-<!DOCTYPE html>
-<head>
-<title>Upgrade Insecure Requests: IFrames.</title>
-<script src="/resources/testharness.js"></script>
-<script src="/resources/testharnessreport.js"></script>
-<meta http-equiv="Content-Security-Policy" content="upgrade-insecure-requests">
-<meta http-equiv="Content-Security-Policy" content="frame-src https:">
-</head>
-<body>
-<script>
-async_test(t => {
- var i = document.createElement('iframe');
-
- // This is a bit of a hack. UPGRADE doesn't upgrade the port number, so we
- // specify this non-existent URL ('http' over port 8443). If UPGRADE doesn't
- // work, it won't load.
- i.src = "http://127.0.0.1:8443/security/resources/post-origin-to-parent.html";
-
- window.addEventListener('message', t.step_func(e => {
- if (e.source == i.contentWindow) {
- assert_equals("https://127.0.0.1:8443", e.data.origin);
- t.done();
- }
- }));
- document.body.appendChild(i);
-}, "Same-host frames are upgraded, and do not violate CSP.");
-
-async_test(t => {
- var i = document.createElement('iframe');
-
- // This is a bit of a hack. UPGRADE doesn't upgrade the port number, so we
- // specify this non-existent URL ('http' over port 8443). If UPGRADE doesn't
- // work, it won't load.
- i.src = "http://example.test:8443/security/resources/post-origin-to-parent.html";
-
- window.addEventListener('message', t.step_func(e => {
- if (e.source == i.contentWindow) {
- assert_equals("https://example.test:8443", e.data.origin);
- t.done();
- }
- }));
- document.body.appendChild(i);
-}, "Cross-host frames are upgraded, and do not violate CSP.");
-
-</script>
-</body>

Powered by Google App Engine
This is Rietveld 408576698