Bug 37143 - Chromium: worker exceptions are not shown in console
Summary: Chromium: worker exceptions are not shown in console
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P3 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-06 02:33 PDT by Andrey Kosyakov
Modified: 2010-04-06 03:14 PDT (History)
2 users (show)

See Also:


Attachments
Fixed reporting of unhandled worker exceptions by inverting check for EventTarget.dispatchEvent result in accordance with its spec (1.52 KB, patch)
2010-04-06 03:01 PDT, Andrey Kosyakov
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Kosyakov 2010-04-06 02:33:11 PDT
workerExceptions.html:

<html>
<script>
var w = new Worker("workerException.js");
</script>
</html>

workerException.js:
throw { };

EXPECTED: Loading workerException.html should cause a console message "Uncaught #<an Object>" to be printed to console.
ACTUAL: nothing happens

This stopped working after r56402, when semantics of EventTarget.dispatchEvent() was slightly changed. The code in WebWorkerClientImpl::postExceptionToWorkerObject(), however, was incorrect before, as EventTarget.dispatchEvent() is supposed to return true iff no event handlers called preventDefault() (i.e. event is unhandled).
Comment 1 Andrey Kosyakov 2010-04-06 03:01:32 PDT
Created attachment 52619 [details]
Fixed reporting of unhandled worker exceptions by inverting check for EventTarget.dispatchEvent result in accordance with its spec
Comment 2 Yury Semikhatsky 2010-04-06 03:03:48 PDT
Comment on attachment 52619 [details]
Fixed reporting of unhandled worker exceptions by inverting check for EventTarget.dispatchEvent result in accordance with its spec

Please make sure this change doesn't break layout tests.
Comment 3 WebKit Commit Bot 2010-04-06 03:14:11 PDT
Comment on attachment 52619 [details]
Fixed reporting of unhandled worker exceptions by inverting check for EventTarget.dispatchEvent result in accordance with its spec

Clearing flags on attachment: 52619

Committed r57138: <http://trac.webkit.org/changeset/57138>
Comment 4 WebKit Commit Bot 2010-04-06 03:14:15 PDT
All reviewed patches have been landed.  Closing bug.