Uploaded a new patch set. The rest of the comments are already dealt with in the
previous patch set.

https://codereview.chromium.org/249503002/diff/40001/src/debug.cc
File src/debug.cc (right):

https://codereview.chromium.org/249503002/diff/40001/src/debug.cc#newcode2700
src/debug.cc:2700: promise = isolate_->factory()->undefined_value();
On 2014/04/24 11:04:59, aandrey wrote:
maybe set undefined to promise by default instead of null? then this
line will
go away naturally

Done.

https://codereview.chromium.org/249503002/

--
--
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.

Reply via email to