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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/preload/preload-video-cors.html

Issue 2476943002: Ensure video poster preloads use the crossorigin attribute (Closed)
Patch Set: Remove empty script block (trybots previous) Created 4 years, 1 month 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/http/tests/preload/preload-video-cors.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/preload/preload-video-cors.html b/third_party/WebKit/LayoutTests/http/tests/preload/preload-video-cors.html
new file mode 100644
index 0000000000000000000000000000000000000000..b9b8b11974f0d0a7a1d9afb360754558eb4fed37
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/http/tests/preload/preload-video-cors.html
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+<script src="../resources/testharness.js"></script>
+<script src="../resources/testharnessreport.js"></script>
+<script>
+ var t = async_test("Ensure preloads use video poster crossorigin value");
+ window.addEventListener('load', t.step_func(function() {
+ var entries = performance.getEntriesByName("http://localhost:8080/security/resources/image-access-control.php?file=../../resources/square100.png&allow=false");
+ assert_equals(entries.length, 0);
Yoav Weiss 2016/11/04 16:07:37 Not sure I get what this test is checking.
Charlie Harrison 2016/11/04 16:11:32 I was trying to check that we never successfully d
Yoav Weiss 2016/11/04 16:45:30 OK. Can you also add tests for: * the same crossor
Charlie Harrison 2016/11/04 17:20:16 I've added the tests in this file. Your rant is no
+ t.done();
+ }));
+</script>
+<video poster="http://localhost:8080/security/resources/image-access-control.php?file=../../resources/square100.png&allow=false" crossorigin>
+</video>
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698