[Webkit-unassigned] [Bug 170395] [Debug] ASSERT(!throwScope.exception()) on imported/w3c/web-platform-tests/fetch/api/cors/cors-preflight-status.html

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 4 10:12:20 PDT 2017


https://bugs.webkit.org/show_bug.cgi?id=170395

--- Comment #8 from youenn fablet <youennf at gmail.com> ---
(In reply to Mark Lam from comment #7)
> (In reply to youenn fablet from comment #6)
> > executeCall is hitting a throwScope.exception() in executeCall.
> > In DeferredPromise::reject, we create a DOM exception which might generate
> > such thing.
> > It might be that the creation of the exception is not happening correctly
> > when we are closing abruptly the Worker.
> 
> Shouldn't DeferredPromise::reject() catch all exceptions?

We could make DeferredPromise::reject() exit early (and not call executeCall) when creation of the JSValue is not going right and is making throwScope.exception() to be true.
I would think that this is only happening in the context of Workers though.
We added several tweaks specific to Workers (see createDeferredPromise for instance).

In Window environment, we should then just ASSERT() while in Worker environment we should return early.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170404/07e7a1e6/attachment.html>


More information about the webkit-unassigned mailing list