(In reply to Chris Pearce (:cpearce) from comment #29)
> @@ +391,1 @@
> > return nsBaseAppShell::Init();
>
> You should probably remove the listener during shutdown too.
Shouldn't be neccessary. The WakeLockListener doesn't depend on anything else being alive, and as jlebar noted, it will be destroyed with the power manager service anyway.
(In reply to Chris Pearce (:cpearce) from comment #29) :Init() ;
> @@ +391,1 @@
> > return nsBaseAppShell:
>
> You should probably remove the listener during shutdown too.
Shouldn't be neccessary. The WakeLockListener doesn't depend on anything else being alive, and as jlebar noted, it will be destroyed with the power manager service anyway.