LGTM!
http://codereview.chromium.org/7129002/diff/1/test/cctest/test-lockers.cc File test/cctest/test-lockers.cc (right): http://codereview.chromium.org/7129002/diff/1/test/cctest/test-lockers.cc#newcode616 test/cctest/test-lockers.cc:616: v8::Locker l(isolate); l -> lock? http://codereview.chromium.org/7129002/diff/1/test/cctest/test-lockers.cc#newcode617 test/cctest/test-lockers.cc:617: v8::Isolate::Scope iscope(isolate); iscope -> isolate_scope to match the other naming? http://codereview.chromium.org/7129002/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
