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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/security/mixedContent/resources/frame-with-insecure-eventsource.html

Issue 1940593002: Make sure the test finishes after EventSource loading has completed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added Access-Control-Allow-Origin http header to the test event stream. Created 4 years, 8 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/mixedContent/resources/frame-with-insecure-eventsource.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/mixedContent/resources/frame-with-insecure-eventsource.html b/third_party/WebKit/LayoutTests/http/tests/security/mixedContent/resources/frame-with-insecure-eventsource.html
index b71065c3b0bf343f22edadf2b3c9859c7a704a86..58bb46b48607821a38873c0dcc2413e51e2062af 100644
--- a/third_party/WebKit/LayoutTests/http/tests/security/mixedContent/resources/frame-with-insecure-eventsource.html
+++ b/third_party/WebKit/LayoutTests/http/tests/security/mixedContent/resources/frame-with-insecure-eventsource.html
@@ -1,8 +1,7 @@
<script>
-var es = new EventSource("http://127.0.0.1:8000/eventsource/resources/simple-event-stream.asis");
-
-window.onload = function() {
+var es = new EventSource("http://127.0.0.1:8000/security/mixedContent/resources/simple-event-stream.pl");
+es.addEventListener("error", function() {
if (window.opener)
window.opener.postMessage('done', '*');
-};
+});
</script>

Powered by Google App Engine
This is Rietveld 408576698