One comment, otherwise LGTM! Thanks, -Ivan
http://codereview.chromium.org/58016/diff/1/2 File src/api.cc (right): http://codereview.chromium.org/58016/diff/1/2#newcode2085 Line 2085: if (js_object->IsUndefined()) { This test seems redundant. hidden_props has to be !undefined to reach here, and the cast from i::Object to i::JSObject is definitely not going to make an undefined value. http://codereview.chromium.org/58016 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---