Store JSGlobalProxy's identity hash directly on the proxy itself

src/object-observe.js

Index: src/object-observe.js
diff --git a/src/object-observe.js b/src/object-observe.js
index c4ddd181d2841c479c3de34bd6d683554e2352bd..e822f0bd497588715e70c3300737b9e72a8e9134 100644
--- a/src/object-observe.js
+++ b/src/object-observe.js
@@ -80,9 +80,13 @@ function GetWeakMapWrapper() {
 
   MapWrapper.prototype = {
     get: function(key) {
+      key = %UnwrapGlobalProxy(key);

[Toon Verwaest, 2014/04/28 14:32:50]

I'd strongly prefer merging UnwrapGlobalProxy with WeakCollectionGet/Set in
"WeakCollectionGetForObserve", so we don't have a free-floating runtime function
that unwraps global proxies, but keep that behavior very local to
Object.observe.

[adamk, 2014/04/28 18:44:50]

Done, factoring out the logic into WeakCollectionGet and WeakCollectionSet
static functions.

+      if (!IS_SPEC_OBJECT(key)) return UNDEFINED;
       return %WeakCollectionGet(this.map_, key);
     },
     set: function(key, value) {
+      key = %UnwrapGlobalProxy(key);
+      if (!IS_SPEC_OBJECT(key)) return UNDEFINED;
       %WeakCollectionSet(this.map_, key, value);
     },
     has: function(key) {