LGTM.
http://codereview.chromium.org/7740021/diff/1/src/accessors.cc File src/accessors.cc (right): http://codereview.chromium.org/7740021/diff/1/src/accessors.cc#newcode752 src/accessors.cc:752: potential_caller != NULL && potential_caller->IsBuiltin(); I think this is clearer as a while loop, or anything that puts the assignments to potential_caller and to caller in the correct order (the body and iteration step). http://codereview.chromium.org/7740021/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
