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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.insecure-parent-frame.html

Issue 2869093002: Upstream Service Worker window test to WPT
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/http/tests/serviceworker/chromium.insecure-parent-frame.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/insecure-parent-frame.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.insecure-parent-frame.html
similarity index 86%
rename from third_party/WebKit/LayoutTests/http/tests/serviceworker/insecure-parent-frame.html
rename to third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.insecure-parent-frame.html
index 98cdf66ffe3eb7fd27d78d3654bd89e218f06540..6aee5c18fdb3c00471412c869c1396c3bd7a2845 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/insecure-parent-frame.html
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.insecure-parent-frame.html
@@ -1,4 +1,10 @@
<!DOCTYPE html>
+<!-- This test's file name is prefixed with `chromium.` because while a roughly
+ equivalent version is available in the Web Platform Tests project, the
+ limitations of the WPT infrastructure precludes the use of iframes for this
+ purpose. This version should be maintained only insofar as the functionality
+ under test concerns iframe contexts specifically; otherwise, modifications
+ should be contributed to the shared version. -->
<meta charset="utf-8">
<script src="../resources/testharness.js"></script>
<script src="../resources/testharnessreport.js"></script>
@@ -40,7 +46,7 @@ promise_test(t => {
var scope = 'resources/insecure-inscope';
var registration;
var insecure_url = get_host_info().UNAUTHENTICATED_ORIGIN +
- '/serviceworker/resources/insecure-parent.html';
+ '/serviceworker/resources/insecure-parent-iframe.html';
var pre_registration_frame;
var post_registration_frame;

Powered by Google App Engine
This is Rietveld 408576698