event callbacks keep getting called. this included key presses, a11y events, and other DOM events we add listeners to when first calling enable().
Created attachment 620933 [details] [diff] [review] shut down AccessFu properly on disable(). I was just being really sloppy in the detach() and enable methods..
Comment on attachment 620933 [details] [diff] [review] shut down AccessFu properly on disable(). OK r=me. I had to refresh myself (via nsIDOMEventTarget.idl comments). Basically you have to add the usecapture arg since you add the event listeners with capture true right? Also the (haha!) s/addObserver/removeObserver should make things nicer :)
yes. you need the usecapture flag. thanks!
http://hg.mozilla.org/mozilla-central/rev/b58c6e5156d7 Should this have a test?