[Pkg-mozext-commits] [firebug] 28/56: Make wrapper.js's cloneFunction do what it's meant to do

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository firebug.

commit 7c29d0188b66d2578795d484ac731e8ba9e6acb0
Author: Simon Lindholm <simon.lindholm10 at gmail.com>
Date:   Wed Oct 22 22:55:28 2014 +0200

    Make wrapper.js's cloneFunction do what it's meant to do
    
    It was completely broken before, only we didn't notice it because of
    transparent chrome object wrappers. Related to issue 7687.
---
 extension/content/firebug/lib/wrapper.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/extension/content/firebug/lib/wrapper.js b/extension/content/firebug/lib/wrapper.js
index e4c2586..0d45e13 100644
--- a/extension/content/firebug/lib/wrapper.js
+++ b/extension/content/firebug/lib/wrapper.js
@@ -128,7 +128,8 @@ function isPrimitive(obj)
 function cloneFunction(win, func)
 {
     var obj = XPCNativeWrapper.unwrap(new win.Object());
-    Object.defineProperty(obj, "f", {value: func});
+    var desc = {value: func, writable: true, configurable: true, enumerable: true};
+    Object.defineProperty(obj, "f", desc);
     Cu.makeObjectPropsNormal(obj);
     return obj.f;
 }

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/firebug.git