LGTM with comments.
https://chromiumcodereview.appspot.com/9415010/diff/1/src/collection.js File src/collection.js (right): https://chromiumcodereview.appspot.com/9415010/diff/1/src/collection.js#newcode28 src/collection.js:28: "use strict"; Some of the .js files have "use strict", some don't. Intentional? https://chromiumcodereview.appspot.com/9415010/diff/1/src/mirror-debugger.js File src/mirror-debugger.js (right): https://chromiumcodereview.appspot.com/9415010/diff/1/src/mirror-debugger.js#newcode27 src/mirror-debugger.js:27: Why new empty line? https://chromiumcodereview.appspot.com/9415010/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
