On 2010-12-24, at 21:30, [email protected] wrote:
> - this.__media.addEventListener('webkitbeginfullscreen', function(evt) {
> self.onbeginfullscreen.sendEvent() }, false);
> - this.__media.addEventListener('webkitendfullscreen', function(evt) {
> self.onendfullscreen.sendEvent() }, false);
> + this.__media.addEventListener('webkitbeginfullscreen', function(evt) {
> this.onbeginfullscreen.sendEvent() }, false);
> + this.__media.addEventListener('webkitendfullscreen', function(evt) {
> this.onendfullscreen.sendEvent() }, false);
This change looks wrong to me. `this` in the closure is going to refer to the
closure's context, not the LZX instance.
The other way to do this would be to uncomment Function.prototype.bind in
LzRuntime (see LPP-8961) and say:
this.__media.addEventListener('webkitendfullscreen', (function(evt) {
this.onendfullscreen.sendEvent() }).bind(this), false);
