divyang4481 / firebreath

Automatically exported from code.google.com/p/firebreath
0 stars 0 forks source link

support unregister event #51

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Brief Description:
if we pass a null value to a event property,it should unregister old event.

Use case:

plugin.onLoad = function() {
    alert("onload")
}

//should unregister event
plugin.onLoad = null;

Additional information:
I have modify two file too support this feature.

Original issue reported on code.google.com by qifuren1...@gmail.com on 30 Jun 2010 at 5:12

Attachments:

GoogleCodeExporter commented 8 years ago
Please note that a better way to submit patches is by putting your changes in a 
clone:

http://groups.google.com/group/firebreath-dev/browse_thread/thread/adfbe6fc6b3d9
aa7

Don't get me wrong; we'll accept them either way =] it just takes longer to 
review them this way. I'll look at it later today and assuming everything looks 
good, I'll pull your changes into the repository.  If before then you decide to 
put them in a branch, I wouldn't complain about that, but no big worries.

Original comment by taxilian on 30 Jun 2010 at 1:03

GoogleCodeExporter commented 8 years ago
This fix should go into a build of FireBreath soon; however, please note that 
the supplied patches were incomplete (though still appreciated) because the 
code to allow the behavior in ActiveX was missing.

Original comment by taxilian on 14 Jul 2010 at 4:26

GoogleCodeExporter commented 8 years ago

Original comment by taxilian on 12 Aug 2010 at 1:46