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

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/preload/single-download-preload.html

Issue 2697453005: Import wpt@758b3b4cfa805067f36121333ba031e583d3a62c (Closed)
Patch Set: Add -expected.txt files. Created 3 years, 10 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/preload/single-download-preload.html
diff --git a/third_party/WebKit/LayoutTests/external/wpt/preload/single_download_preload.html b/third_party/WebKit/LayoutTests/external/wpt/preload/single-download-preload.html
similarity index 67%
rename from third_party/WebKit/LayoutTests/external/wpt/preload/single_download_preload.html
rename to third_party/WebKit/LayoutTests/external/wpt/preload/single-download-preload.html
index 0c20194b09b77e0fafee2e7af651f08ed3ef4f2f..c06e3e57833802163bc88273733169319a5093e2 100644
--- a/third_party/WebKit/LayoutTests/external/wpt/preload/single_download_preload.html
+++ b/third_party/WebKit/LayoutTests/external/wpt/preload/single-download-preload.html
@@ -1,6 +1,7 @@
<!DOCTYPE html>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
+<script src="/preload/resources/preload_helper.js"></script>
<script>
var t = async_test('Makes sure that preloaded resources are not downloaded again when used');
</script>
@@ -42,9 +43,19 @@
xhr.send();
window.addEventListener("load", t.step_func(function() {
- // Audio and video show 2 extra requests as the main request is followed by a range request
- assert_equals(performance.getEntriesByType("resource").length, 14);
- t.done();
+ verifyPreloadAndRTSupport();
+ setTimeout(t.step_func(function() {
+ verifyNumberOfDownloads("resources/dummy.js", 1);
+ verifyNumberOfDownloads("resources/dummy.css", 1);
+ verifyNumberOfDownloads("resources/square.png", 1);
+ verifyNumberOfDownloads("resources/square.png?background", 1);
+ verifyNumberOfDownloads("/fonts/CanvasTest.ttf", 1);
+ verifyNumberOfDownloads("resources/dummy.xml?foobar", 0);
+ verifyNumberOfDownloads("/media/foo.vtt", 1);
+ verifyNumberOfDownloads("resources/dummy.xml", 1);
+ // FIXME: We should verify for video and audio as well, but they seem to (flakily?) trigger multiple partial requests.
+ t.done();
+ }), 0);
}));
</script>
<span>PASS - this text is here just so that the browser will download the font.</span>

Powered by Google App Engine
This is Rietveld 408576698