https://codereview.chromium.org/416213004/diff/60001/src/promise.js
File src/promise.js (right):
https://codereview.chromium.org/416213004/diff/60001/src/promise.js#newcode61
src/promise.js:61: %DebugPromiseEvent({ promise: promise,
nit: can it be a one liner now?
https://codereview.chromium.org/416213004/diff/60001/src/promise.js#newcode238
src/promise.js:238: %DebugPromiseEvent({ promise: deferred.promise,
ditto
https://codereview.chromium.org/416213004/diff/60001/test/mjsunit/es6/debug-promise-events.js
File test/mjsunit/es6/debug-promise-events.js (right):
https://codereview.chromium.org/416213004/diff/60001/test/mjsunit/es6/debug-promise-events.js#newcode44
test/mjsunit/es6/debug-promise-events.js:44:
assertTrue(event_data.promise().isPromise());
code dup x3
https://codereview.chromium.org/416213004/diff/60001/test/mjsunit/es6/debug-promise-events.js#newcode64
test/mjsunit/es6/debug-promise-events.js:64: var p4 = p3.then();
there is still no tree of promises, isn't it? maybe test this also?
https://codereview.chromium.org/416213004/diff/60001/test/mjsunit/es6/debug-promise-events.js#newcode66
test/mjsunit/es6/debug-promise-events.js:66: function assertOrAbort(b,
s) {
can we use same name from where it's copied, to ease code search
https://codereview.chromium.org/416213004/diff/60001/test/mjsunit/es6/debug-promise-events.js#newcode78
test/mjsunit/es6/debug-promise-events.js:78: function() {
extract this function to top level
https://codereview.chromium.org/416213004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.