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

Unified Diff: LayoutTests/fast/workers/worker-constructor-expected.txt

Issue 84423002: Correctly resolve empty script URLs in Worker constructors. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 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
Index: LayoutTests/fast/workers/worker-constructor-expected.txt
diff --git a/LayoutTests/fast/workers/worker-constructor-expected.txt b/LayoutTests/fast/workers/worker-constructor-expected.txt
index 7c0555b6fe2f77cca5e7a1cf9b1430ca29e4e569..bb7b7d20d2ea081898f70ef291edf7f836f721f5 100644
--- a/LayoutTests/fast/workers/worker-constructor-expected.txt
+++ b/LayoutTests/fast/workers/worker-constructor-expected.txt
@@ -3,7 +3,7 @@ Test Worker constructor functionality. Should print a series of PASS messages, f
PASS: toString exception propagated correctly.
PASS: trying to create workers recursively resulted in an exception (RangeError: Maximum call stack size exceeded)
PASS: invoking Worker constructor without arguments resulted in an exception (TypeError: Failed to execute 'Constructor' on 'Worker': 1 argument required, but only 0 present.)
-PASS: invoking Worker constructor with empty script URL resulted in an exception (SyntaxError: Failed to create a worker: an empty URL was provided.)
+PASS: onerror invoked for an empty script URL, resolving to this HTML document's URL.
PASS: invoking Worker constructor with invalid script URL resulted in an exception (SyntaxError: Failed to create a worker: 'http://invalid:123$' is not a valid URL.)
PASS: onerror invoked for a script that could not be loaded.
PASS: Successfully created worker.
« no previous file with comments | « LayoutTests/fast/workers/worker-constructor.html ('k') | LayoutTests/http/tests/workers/shared-worker-invalid-url.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698