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

Unified Diff: LayoutTests/http/tests/workers/worker-importScripts-onerror-redirect-to-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: 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-redirect-to-crossorigin.html
diff --git a/LayoutTests/http/tests/workers/worker-importScripts-onerror-redirect-to-crossorigin.html b/LayoutTests/http/tests/workers/worker-importScripts-onerror-redirect-to-crossorigin.html
index e918feaed359b2cafa00db582561e69e363c5fe9..f78409ce9fd78f3f610ca33a08ee743260dc5ff0 100644
--- a/LayoutTests/http/tests/workers/worker-importScripts-onerror-redirect-to-crossorigin.html
+++ b/LayoutTests/http/tests/workers/worker-importScripts-onerror-redirect-to-crossorigin.html
@@ -14,28 +14,28 @@
var worker = new Worker('resources/worker-importscripts-onerror-redirect-to-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-redirect-to-crossorigin.js");
- shouldEvaluateTo("workerOnerror.lineno", 6);
+ shouldBeEqualToString("workerOnerror.message", "Script error.");
+ shouldBeEqualToString("workerOnerror.filename", "");
+ shouldEvaluateTo("workerOnerror.lineno", 0);
shouldEvaluateTo("workerOnerror.column", 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-redirect-to-crossorigin.js");
- shouldEvaluateTo("onerrorLine", 6);
+ pageOnerror = e;
+ shouldBeEqualToString("pageOnerror.message", "Script error.");
+ shouldBeEqualToString("pageOnerror.filename", "");
+ shouldEvaluateTo("pageOnerror.lineno", 0);
+ shouldEvaluateTo("pageOnerror.colno", 0);
+ shouldBeNull("pageOnerror.error");
e.preventDefault();
finishJSTest();
};
</script>
- <script src="/resources/js-test-post.js"></script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698