I don't see any place where you remove this.  Does that mean we're leaking an event handler at some point?