Yeah, I can see pros and cons to both approaches.  Maybe it's best to stick with what we normally do in the rest of Cesium and keep the boring, stupid, consistent, faster `var that = this;` style.