If you're not add/RemoveRef()'ing anymore then that should work fine.  Lots of stuff relies on activeEditorChange, so I think we can trust the event to fire reliably.  That said, the code as written here should work fine too.