LGTM (with the one comment addressed!)
http://codereview.chromium.org/9178021/diff/1/src/spaces.cc File src/spaces.cc (right): http://codereview.chromium.org/9178021/diff/1/src/spaces.cc#newcode656 src/spaces.cc:656: if (!chunk->InNewSpace()) { We need an additional check here to make sure that the page was elected for lazy sweeping and isn't just being incrementally marked. http://codereview.chromium.org/9178021/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
