LGTM

http://codereview.chromium.org/390001/diff/2001/3002
File src/runtime.cc (right):

http://codereview.chromium.org/390001/diff/2001/3002#newcode6026
Line 6026: Object* e = old_names->get(i);
Rename 'e' to 'name'?

http://codereview.chromium.org/390001/diff/2001/3001
File test/cctest/test-debug.cc (right):

http://codereview.chromium.org/390001/diff/2001/3001#newcode3442
Line 3442: // See http://crbug.com/26491
Could you add a one line description before the bug link?

http://codereview.chromium.org/390001

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

Reply via email to