On Mon, Jun 22, 2009 at 10:45, <[email protected]> wrote: > > http://codereview.chromium.org/131107/diff/1/2 > File src/mirror-delay.js (right): > > http://codereview.chromium.org/131107/diff/1/2#newcode2241 > Line 2241: ref: scope.scopeObject().handle() > On 2009/06/22 06:42:28, Søren Gjesse wrote: > >> For transient scope objects (negative handles) you should skip the ref >> > to the > >> handle, as it cannot be retrieved through that id anyway. >> > What do you mean by 'it cannot be retrieved through that id', the scope > object mirror is put into the mirror cache in MakeMirror? > > > http://codereview.chromium.org/131107 >
If you look at ScopeMirror.prototype.scopeObject in mirror-delay.js ScopeType.Local and ScopeType.Closure are created as transient mirrors which are not added to the mirror cache. The reason for this is that these scope objects does not exist in the VM but are created by the debugger when required. As the data from which these objects where created can change caching them in the mirror cache would lead to stale data. --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
