Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.fetch-request-xhr.html |
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/fetch-request-xhr.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.fetch-request-xhr.html |
similarity index 70% |
rename from third_party/WebKit/LayoutTests/http/tests/serviceworker/fetch-request-xhr.html |
rename to third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.fetch-request-xhr.html |
index 85dbe4c9cb9a04b693817e62e1b383b7224691b2..133e7b4a88c2de8d5faf1ed2cc20fc7261bfd283 100644 |
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/fetch-request-xhr.html |
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.fetch-request-xhr.html |
@@ -1,4 +1,14 @@ |
<!DOCTYPE html> |
+<!-- This test is prefixed with `chromium.` because the equivalent version |
+ available in Web Platform Tests is known to cause timeout errors in the |
+ Chromium automated build system. See https://crbug.com/658997 |
+ |
+ Furthermore, the upstream version is more strict, and Chromium currently |
+ fails it. See https://crbug.com/595993 |
+ |
+ This version should be maintained only to preserve test coverage until the |
+ corresponding version in Web Platform Tests can be made to pass consistently. |
+ --> |
<title>Service Worker: the body of FetchEvent using XMLHttpRequest</title> |
<script src="../resources/testharness.js"></script> |
<script src="../resources/testharnessreport.js"></script> |
@@ -15,10 +25,10 @@ async_test(function(t) { |
}) |
.then(function() { return with_iframe(SCOPE); }) |
.then(function(frame) { |
+ t.add_cleanup(function() { frame.remove(); }); |
var channel = new MessageChannel(); |
channel.port1.onmessage = t.step_func(function(e) { |
assert_equals(e.data.results, 'finish'); |
- frame.remove(); |
service_worker_unregister_and_done(t, SCOPE); |
}); |
frame.contentWindow.postMessage({}, |