On 2012/04/26 14:31:15, ulan wrote:
This looks good to me if you remove the trailing space.
Please wait for Yang approval of changes in src/mirror-debugger.js and
src/debug-debugger.js before landing.
https://chromiumcodereview.appspot.com/10171003/diff/26001/test/mjsunit/harmony/debug-function-scopes.js
File test/mjsunit/harmony/debug-function-scopes.js (right):
https://chromiumcodereview.appspot.com/10171003/diff/26001/test/mjsunit/harmony/debug-function-scopes.js#newcode82
test/mjsunit/harmony/debug-function-scopes.js:82: var f2 = (function() {
Trailing space.
LGTM as well.
http://codereview.chromium.org/10171003/
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
