Drive-by-comments: As discussed offline with Andreas, I think we will need a
separate "InternalWeakMap" that builds on the same backing functionality as
WeakMaps but isn't bound to a specific context. Unfortunately I am not yet sure
about the specifics of that.


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

https://codereview.chromium.org/11274014/diff/14001/src/object-observe.js#newcode48
src/object-observe.js:48: objectObservationState.observerInfoMap =
createInternalWeakMap();
This will make the WeakMap instantiated in this context globally
reachable from the whole isolate. Thereby the first context will be kept
alive indefinitely.

https://codereview.chromium.org/11274014/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to