https://codereview.chromium.org/154283002/diff/180001/include/v8.h
File include/v8.h (right):
https://codereview.chromium.org/154283002/diff/180001/include/v8.h#newcode5421
include/v8.h:5421: static const int kEmptyStringRootIndex = 148;
did you run make grokdump and commit the changes?
https://codereview.chromium.org/154283002/diff/180001/src/api.cc
File src/api.cc (right):
https://codereview.chromium.org/154283002/diff/180001/src/api.cc#newcode6315
src/api.cc:6315: ExternalMicrotaskCallback callback) {
No need for a new ExternalMicrotaskCallback type: just use
Handle<Function> here as the second argument.
Also, I don't see a reason to call all these things ExternalMicrotask
everywhere, just Microtask is maybe better.
https://codereview.chromium.org/154283002/
--
--
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/groups/opt_out.
