LGTM
https://chromiumcodereview.appspot.com/11299248/diff/1/src/global-handles.cc File src/global-handles.cc (right): https://chromiumcodereview.appspot.com/11299248/diff/1/src/global-handles.cc#newcode567 src/global-handles.cc:567: if (!f(isolate_->heap(), &object)) { Maybe rename "f" to something more descriptive like "CanSkipObject"? https://chromiumcodereview.appspot.com/11299248/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
