LGTM.
http://codereview.chromium.org/7828080/diff/1/src/bootstrapper.cc File src/bootstrapper.cc (right): http://codereview.chromium.org/7828080/diff/1/src/bootstrapper.cc#newcode1324 src/bootstrapper.cc:1324: INSTALL_NATIVE(JSFunction, "ToPropertyDescriptor", to_property_descriptor); Do you need this one? http://codereview.chromium.org/7828080/diff/1/src/objects.cc File src/objects.cc (right): http://codereview.chromium.org/7828080/diff/1/src/objects.cc#newcode2313 src/objects.cc:2313: Handle<String> enum_name = isolate->factory()->LookupAsciiSymbol("enumerable"); It looks like there are a couple of long lines. Check presubmit before committing. http://codereview.chromium.org/7828080/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
