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

Unified Diff: third_party/WebKit/LayoutTests/fast/workers/dedicated-worker-error-event.html

Issue 2804533002: Update Error Event inside a worker to provide the exact exception value (Closed)
Patch Set: fix nits Created 3 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/fast/workers/dedicated-worker-error-event.html
diff --git a/third_party/WebKit/LayoutTests/fast/workers/dedicated-worker-error-event.html b/third_party/WebKit/LayoutTests/fast/workers/dedicated-worker-error-event.html
new file mode 100644
index 0000000000000000000000000000000000000000..f076651ea19437a1e2bd2a3ecea7f73dddf1de44
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/workers/dedicated-worker-error-event.html
@@ -0,0 +1,16 @@
+<!doctype html>
+<meta charset=utf-8>
+<title>Dedicated Worker: Error event error message</title>
+<script src=../../resources/testharness.js></script>
+<script src=../../resources/testharnessreport.js></script>
+<script>
+promise_test(function(t) {
nhiroki 2017/04/06 01:50:35 Can you add a comment like this? // TODO(yiyix):
yiyix 2017/04/06 04:38:19 Done.
+ return new Promise(function(resolve) {
nhiroki 2017/04/06 01:50:35 Optional: Arrow function would be simpler: return
yiyix 2017/04/06 04:38:19 Done.
+ var worker = new Worker('resources/error-script.js');
+ worker.onmessage = resolve;
+ })
nhiroki 2017/04/06 01:50:35 +2 indents for line 8-11 return new Promise(r
yiyix 2017/04/06 04:38:19 Done.
+ .then(function(e) {
+ assert_equals(e.data.value, 'testError');
+ });
+ }, 'Error handlers inside a worker should see the error value');
+</script>

Powered by Google App Engine
This is Rietveld 408576698