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

Unified Diff: LayoutTests/http/tests/workers/worker-importScripts-onerror-crossorigin.html

Issue 22467005: Correctly attribute exceptions thrown inside a Worker's imported scripts. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Killing windows-specific expectations. Created 7 years, 4 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: LayoutTests/http/tests/workers/worker-importScripts-onerror-crossorigin.html
diff --git a/LayoutTests/http/tests/workers/worker-importScripts-onerror-crossorigin.html b/LayoutTests/http/tests/workers/worker-importScripts-onerror-crossorigin.html
index 449317b109e69688481d38b5aa91d237ef0a06c5..a8921751ff60f8c8e12a274efe0e20427dc12701 100644
--- a/LayoutTests/http/tests/workers/worker-importScripts-onerror-crossorigin.html
+++ b/LayoutTests/http/tests/workers/worker-importScripts-onerror-crossorigin.html
@@ -14,23 +14,24 @@
var worker = new Worker('resources/worker-importscripts-onerror-crossorigin.js');
- var workerOnerror;
+ var workerOnerror, pageOnerror;
worker.onmessage = function (e) {
workerOnerror = e.data;
- shouldBeEqualToString("workerOnerror.message", "Uncaught undefined");
- shouldBeEqualToString("workerOnerror.filename", "http://127.0.0.1:8000/workers/resources/worker-importscripts-onerror-crossorigin.js");
- shouldEvaluateTo("workerOnerror.lineno", 5);
- shouldEvaluateTo("workerOnerror.column", 0);
+ shouldBeEqualToString("workerOnerror.message", "Script error.");
+ shouldBeEqualToString("workerOnerror.filename", "");
+ shouldEvaluateTo("workerOnerror.lineno", 0);
+ shouldEvaluateTo("workerOnerror.colno", 0);
+ shouldBeNull("workerOnerror.error");
};
var onerrorMessage, onerrorURL, onerrorLine;
worker.onerror = function (e) {
- onerrorMessage = e.message;
- onerrorURL = e.filename;
- onerrorLine = e.lineno;
- shouldBeEqualToString("onerrorMessage", "Uncaught undefined");
- shouldBeEqualToString("onerrorURL", "http://127.0.0.1:8000/workers/resources/worker-importscripts-onerror-crossorigin.js");
- shouldEvaluateTo("onerrorLine", 5);
+ pageOnerror = e;
+ shouldBeEqualToString("pageOnerror.message", "Script error.");
+ shouldBeEqualToString("pageOnerror.filename", "");
+ shouldEvaluateTo("pageOnerror.lineno", 0);
+ shouldEvaluateTo("pageOnerror.colno", 0);
+ shouldBeNull("pageOnerror.error");
e.preventDefault();
finishJSTest();

Powered by Google App Engine
This is Rietveld 408576698