LGTM (if one nit is addressed).
https://chromiumcodereview.appspot.com/10535011/diff/1/src/objects.cc File src/objects.cc (right): https://chromiumcodereview.appspot.com/10535011/diff/1/src/objects.cc#newcode2555 src/objects.cc:2555: ASSERT(result->type() != INTERCEPTOR); Can we move that out of into one line doing just the assertion. Like this: ASSERT(!result->IsProperty() || result->type() != INTERCEPTOR); https://chromiumcodereview.appspot.com/10535011/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
