LGTM with a comment.

Awesome! Thanks for fixing this!


http://codereview.chromium.org/3311002/diff/9/16007
File test/cctest/test-log-stack-tracer.cc (right):

http://codereview.chromium.org/3311002/diff/9/16007#newcode249
test/cctest/test-log-stack-tracer.cc:249:
ASSERT(frame_iterator.frame()->is_exit());
maybe replace this with a CHECK, so it will abort in release mode, too?

http://codereview.chromium.org/3311002/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to