Reviewers: jarin,
Description:
[d8 Workers] Fix bug creating Worker during main thread termination
When the main thread terminates, it forcibly terminates all Worker threads.
When this happens, the threads objects were only half-created; they had a
JavaScript Worker object, but not a C++ worker object.
This CL fixes that bug, as well as some other fixes:
* Signatures on Worker methods
* Use SetAlignedPointerFromInternalField instead of using an External.
* Don't use atomics in Worker: state_ is only accessed by the parent thread
(the thread that created the Worker).
BUG=chromium:511880
[email protected]
LOG=n
Please review this at https://codereview.chromium.org/1255563002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+64, -46 lines):
M src/d8.h
M src/d8.cc
A test/mjsunit/regress/regress-crbug-511880.js
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
