Bug 1005552 - Stop binding marquee event handlers. r=bz

These functions get invoked as event listeners, so we'll automatically get the
proper |this|. The reason for the existing shenanigans was to work around
bug 872772, which has now been fixed.
This commit is contained in:
Bobby Holley 2014-05-16 10:56:41 -07:00
parent 252c68c45e
commit 0378fc7bf6

View File

@ -266,11 +266,7 @@
// call it (since XBL gets opaque non-callable wrappers to content).
// * The fact that contentFn is transitively waived, which we need
// in order to be able to invoke it.
// * Using the local |bind| to be sure that it does the right thing.
// Note that the underlying function still executes in the content
// scope.
var contentFn = new (XPCNativeWrapper.unwrap(window.Function))("event", aValue);
this["_on" + aName] = Function.prototype.bind.call(contentFn, this);
this["_on" + aName] = new (XPCNativeWrapper.unwrap(window.Function))("event", aValue);
}
catch(e) {
return false;