mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Keep an id for each object added to global weak map.

Browse source
This commit is contained in:
Cheng Zhao 2013-04-26 22:33:31 +08:00
parent cc921fed1e
commit 536b523232
1 changed files with 11 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
browser/atom/objects_registry.coffee
View file

@ -3,6 +3,12 @@ IDWeakMap = require 'id_weak_map'
globalStore = {} globalStore = {}
globalMap = new IDWeakMap globalMap = new IDWeakMap
addObjectToWeakMap = (obj) ->
id = globalMap.add obj
Object.defineProperty obj, 'id',
enumerable: true, writable: false, value: id
id
getStoreForRenderView = (process_id, routing_id) -> getStoreForRenderView = (process_id, routing_id) ->
key = "#{process_id}_#{routing_id}" key = "#{process_id}_#{routing_id}"
globalStore[key] = {} unless globalStore[key]? globalStore[key] = {} unless globalStore[key]?
@ -10,13 +16,15 @@ getStoreForRenderView = (process_id, routing_id) ->
process.on 'ATOM_BROWSER_INTERNAL_NEW', (obj) -> process.on 'ATOM_BROWSER_INTERNAL_NEW', (obj) ->
# For objects created in browser scripts, keep a weak reference here. # For objects created in browser scripts, keep a weak reference here.
id = globalMap.add obj addObjectToWeakMap obj
obj.id = id
exports.add = (process_id, routing_id, obj) -> exports.add = (process_id, routing_id, obj) ->
# Some native types may already been added to globalMap, in that case we # Some native types may already been added to globalMap, in that case we
# don't add it twice. # don't add it twice.
id = obj.id ? globalMap.add obj if obj.id?
id = obj.id
else
id = addObjectToWeakMap obj
store = getStoreForRenderView process_id, routing_id store = getStoreForRenderView process_id, routing_id