lgtm, nits
https://codereview.chromium.org/290633010/diff/1/src/heap.h
File src/heap.h (right):
https://codereview.chromium.org/290633010/diff/1/src/heap.h#newcode186
src/heap.h:186: V(JSObject, observation_state, ObservationState)
\
unrelated, but is this another context leak?
https://codereview.chromium.org/290633010/diff/1/src/isolate.cc
File src/isolate.cc (right):
https://codereview.chromium.org/290633010/diff/1/src/isolate.cc#newcode2273
src/isolate.cc:2273: while (pending_microtask_count() > 0) {
put handlescope here
https://codereview.chromium.org/290633010/diff/1/src/isolate.cc#newcode2280
src/isolate.cc:2280: for (int i = 0; i < num_tasks; i++) {
and here maybe
https://codereview.chromium.org/290633010/
--
--
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.
