Accessing members all the time like this._button is not very efficient, and also pretty inelegant. Keep a |var button = this._button;| and use that in this method instead of this._button everywhere.

Usually as soon as I access 2 times this._something in a method I do a local variable for it.