https://codereview.chromium.org/11266011/diff/16001/src/contexts.h File src/contexts.h (right):
https://codereview.chromium.org/11266011/diff/16001/src/contexts.h#newcode297 src/contexts.h:297: DELIVER_CHANGE_RECORDS_INDEX, Is this perhaps some orphan duplicate of the above? https://codereview.chromium.org/11266011/diff/16001/src/isolate.h File src/isolate.h (right): https://codereview.chromium.org/11266011/diff/16001/src/isolate.h#newcode357 src/isolate.h:357: V(bool, has_active_object_observers, false) \ Maybe put a comment here explaining when this is true (the name is slightly misleading). https://codereview.chromium.org/11266011/diff/16001/src/object-observe.js File src/object-observe.js (right): https://codereview.chromium.org/11266011/diff/16001/src/object-observe.js#newcode116 src/object-observe.js:116: var pendingChangeRecords = observerInfo.pendingChangeRecords; What is this variable used for? https://codereview.chromium.org/11266011/diff/16001/src/object-observe.js#newcode118 src/object-observe.js:118: pendingChangeRecords = observerInfo.pendingChangeRecords = new InternalArray(changeRecord); Line length https://codereview.chromium.org/11266011/diff/16001/src/objects.cc File src/objects.cc (right): https://codereview.chromium.org/11266011/diff/16001/src/objects.cc#newcode1746 src/objects.cc:1746: isolate->set_has_active_object_observers(false); This should probably be go to the end. https://codereview.chromium.org/11266011/diff/16001/src/v8.cc File src/v8.cc (right): https://codereview.chromium.org/11266011/diff/16001/src/v8.cc#newcode228 src/v8.cc:228: if (has_active_object_observers) { FLAG_harmony_observation && ... https://codereview.chromium.org/11266011/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
