Bug 28932
| Summary: | WorkerContext::reportException() should dispatch exceptions to event listeners | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Andrew Wilson <atwilson> |
| Component: | WebCore JavaScript | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | ap |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
Andrew Wilson
WorkerContext::reportException() only passes on the exception to error handlers registered via onerror().
It should also dispatch the error to errors registered via addEventListener().
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Andrew Wilson
Re-reading the spec, I believe this I was incorrect.
Section 4.7 says:
"Whenever an uncaught runtime script error occurs in one of the worker's scripts, if the error did not occur while handling a previous script error, the user agent must report the error using the WorkerGlobalScope object's onerror attribute. [HTML5]"
Once the error has been fired at the onerror attribute, *then* it's turned into an event and sent back to the parent worker (which we correctly do now). But we don't need to fire an event in worker scope.
Resolving this bug as invalid.