I have made this change to get this landed, but I still disagree somewhat with your logic. We bind directly to the mozmill instance on the next line anyway. As I see it, the testrun subclasses are forced to use an alternative method for adding listeners, but there's no reason for that alternative method to be used in the base testrun class.