LGTM with comments
https://chromiumcodereview.appspot.com/10703103/diff/1/src/objects.cc File src/objects.cc (right): https://chromiumcodereview.appspot.com/10703103/diff/1/src/objects.cc#newcode2863 src/objects.cc:2863: // From this point on everything needs to be handlified, because Actually, the point is after the special handling of JSGlobalProxy, which does a recursive call into this same function again. https://chromiumcodereview.appspot.com/10703103/diff/1/src/objects.cc#newcode2865 src/objects.cc:2865: Handle<JSObject> object(this); Can we call it 'self', like in other places? That's a bit more descriptive. https://chromiumcodereview.appspot.com/10703103/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
