LGTM with small nit
http://codereview.chromium.org/8136004/diff/1/test/cctest/test-weakmaps.cc File test/cctest/test-weakmaps.cc (right): http://codereview.chromium.org/8136004/diff/1/test/cctest/test-weakmaps.cc#newcode95 test/cctest/test-weakmaps.cc:95: NumberOfDeletedElements()); I don't like this indention or the fact that we end the line in ->, could we move down the parameters and do 4 space indent, same below http://codereview.chromium.org/8136004/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
