LGTM
http://codereview.chromium.org/7890003/diff/1/test/mjsunit/harmony/weakmaps.js File test/mjsunit/harmony/weakmaps.js (right): http://codereview.chromium.org/7890003/diff/1/test/mjsunit/harmony/weakmaps.js#newcode160 test/mjsunit/harmony/weakmaps.js:160: // Regression test: The parent of the WeakMap prototype is supposed to be the That comment isn't quite in sync with JS terminology. It just repeats the code anyway. I suggest moving the assertion to line 141 and simple dropping the comment. http://codereview.chromium.org/7890003/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
