Support multiple trappers to same property in set-constant

Related issues:
- https://github.com/uBlockOrigin/uBlock-issues/issues/156
- https://github.com/uBlockOrigin/uBlock-issues/issues/1162

Take into account that a trapped property may have been
already trapped, and if so honour previous trapper
getter/setter.

assets/resources/scriptlets.js

@@ -588,8 +588,9 @@
 /// set-constant.js
 /// alias set.js
 (function() {
-    const thisScript = document.currentScript;
+    const chain = '{{1}}';
     let cValue = '{{2}}';
+    const thisScript = document.currentScript;
     if ( cValue === 'undefined' ) {
         cValue = undefined;
     } else if ( cValue === 'false' ) {
@@ -622,53 +623,83 @@
             (typeof v !== typeof cValue);
         return aborted;
     };
-    const makeProxy = function(owner, chain) {
+    // https://github.com/uBlockOrigin/uBlock-issues/issues/156
-        const pos = chain.indexOf('.');
+    //   Support multiple trappers for the same property.
-        if ( pos === -1 ) {
+    const trapProp = function(owner, prop, handler) {
-            const original = owner[chain];
+        if ( handler.init(owner[prop]) === false ) { return; }
-            if ( mustAbort(original) ) { return; }
+        const odesc = Object.getOwnPropertyDescriptor(owner, prop);
-            const desc = Object.getOwnPropertyDescriptor(owner, chain);
+        let prevGetter, prevSetter;
-            if ( desc === undefined || desc.get === undefined ) {
+        if ( odesc instanceof Object ) {
-                Object.defineProperty(owner, chain, {
+            if ( odesc.get instanceof Function ) {
-                    get: function() {
+                prevGetter = odesc.get;
-                        return document.currentScript === thisScript
-                            ? original
-                            : cValue;
-                    },
-                    set: function(a) {
-                        if ( mustAbort(a) ) {
-                            cValue = a;
             }
+            if ( odesc.set instanceof Function ) {
+                prevSetter = odesc.set;
+            }
+        }
+        Object.defineProperty(owner, prop, {
+            configurable: true,
+            get() {
+                if ( prevGetter !== undefined ) {
+                    prevGetter();
+                }
+                return handler.getter();
+            },
+            set(a) {
+                if ( prevSetter !== undefined ) {
+                    prevSetter(a);
+                }
+                handler.setter(a);
             }
         });
+    };
+    const trapChain = function(owner, chain) {
+        const pos = chain.indexOf('.');
+        if ( pos === -1 ) {
+            trapProp(owner, chain, {
+                v: undefined,
+                init: function(v) {
+                    if ( mustAbort(v) ) { return false; }
+                    this.v = v;
+                    return true;
+                },
+                getter: function() {
+                    return document.currentScript === thisScript
+                        ? this.v
+                        : cValue;
+                },
+                setter: function(a) {
+                    if ( mustAbort(a) === false ) { return; }
+                    cValue = a;
                 }
+            });
             return;
         }
         const prop = chain.slice(0, pos);
-        let v = owner[prop];
+        const v = owner[prop];
         chain = chain.slice(pos + 1);
-        if (
+        if ( v instanceof Object || typeof v === 'object' && v !== null ) {
-            (v instanceof Object) ||
+            trapChain(v, chain);
-            (typeof v === 'object' && v !== null)
-        ) {
-            makeProxy(v, chain);
             return;
         }
-        const desc = Object.getOwnPropertyDescriptor(owner, prop);
+        trapProp(owner, prop, {
-        if ( desc && desc.set !== undefined ) { return; }
+            v: undefined,
-        Object.defineProperty(owner, prop, {
+            init: function(v) {
-            get: function() {
+                this.v = v;
-                return v;
+                return true;
             },
-            set: function(a) {
+            getter: function() {
-                v = a;
+                return this.v;
+            },
+            setter: function(a) {
+                this.v = a;
                 if ( a instanceof Object ) {
-                    makeProxy(a, chain);
+                    trapChain(a, chain);
                 }
             }
         });
     };
-    makeProxy(window, '{{1}}');
+    trapChain(window, chain);
 })();