[Object.observe] Lazily allocate callbackInfo structure

[Object.observe] Lazily allocate callbackInfo structure

This patch allows callbacks to lazily allocate the InternalArray which is used to store pendingChangeRecords. This moves some of the expense of observation to the case where changes actually occurred.

When there are no pendingChangeRecords, the callbackInfo structure is a number which is the callbacks priority. Whenever a changeRecord is enqueued to the callback, it "normalizes" to be an InternalArray with a priority property. Immediately before its changeRecords are delivered, it returns to its optimized state.

---
Note: Naming confusion resolved:

This patch corrects some naming confusion in object-observe.js. Previously, we used the terms "callback" and "observer" to mean roughly the same thing, and overloaded the term "observer" to be both the callback itself and the *registration* on a object to observe (which now includes an accept map).

This patch resolves this confusion:

"object" (objectInfo, objectInfoMap): This refers to the observed object and its structures

"callback" (callbackInfo, callbackInfoMap): This refers to the callback to whom change records may be delivered

"observer" (objectInfo.changeObservers): This refers to a registration to observe a given object by a given callback with the specified accept list.
---

R=rossberg@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=15682

[rossberg, 2013-07-15 17:02:43]

LGTM with nit

https://codereview.chromium.org/19132002/diff/1/src/object-observe.js
File src/object-observe.js (right):

https://codereview.chromium.org/19132002/diff/1/src/object-observe.js#newcode206
src/object-observe.js:206: function NormalizeCallback(callback) {
Nit: Not sure I like the name too much, since it's not normalizing the callback
but its info. GetCallbackInfo, perhaps?

[rafaelw, 2013-07-15 19:13:11]

https://codereview.chromium.org/19132002/diff/1/src/object-observe.js
File src/object-observe.js (right):

https://codereview.chromium.org/19132002/diff/1/src/object-observe.js#newcode206
src/object-observe.js:206: function NormalizeCallback(callback) {
Agreed. I'd like to be explicit that this call is "de-optimizing" the
representation. I'm going with NormalizeCallbackInfo.

If you're not happy with this, I can fix it in a follow-up patch.

On 2013/07/15 17:02:43, rossberg wrote:
> Nit: Not sure I like the name too much, since it's not normalizing the
callback
> but its info. GetCallbackInfo, perhaps?

[adamk, 2013-07-15 22:16:35]

Committed patchset #2 manually as r15682 (presubmit successful).