(extracted some explanation that was hidden in comments in other answer)
The problem lies in the following line:
this.dom.addEventListener("click", self.onclick, false);
Here, you pass a function object to be used as callback. When the event trigger, the function is called but now it has no association with any object (this).
The problem can be solved by wrapping the function (with it's object reference) in a closure as follows:
this.dom.addEventListener(
"click",
function(event) {self.onclick(event)},
false);
Since the variable self was assigned this when the closure was created, the closure function will remember the value of the self variable when it's called at a later time.
An alternative way to solve this is to make an utility function (and avoid using variables for binding this):
function bind(scope, fn) {
return function () {
fn.apply(scope, arguments);
};
}
The updated code would then look like:
this.dom.addEventListener("click", bind(this, this.onclick), false);
Function.prototype.bind is part of ECMAScript 5 and provides the same functionality. So you can do:
this.dom.addEventListener("click", this.onclick.bind(this), false);
For browsers which do not support ES5 yet, MDN provides the following shim:
if (!Function.prototype.bind) {
Function.prototype.bind = function (oThis) {
if (typeof this !== "function") {
// closest thing possible to the ECMAScript 5 internal IsCallable function
throw new TypeError("Function.prototype.bind - what is trying to be bound is not callable");
}
var aArgs = Array.prototype.slice.call(arguments, 1),
fToBind = this,
fNOP = function () {},
fBound = function () {
return fToBind.apply(this instanceof fNOP
? this
: oThis || window,
aArgs.concat(Array.prototype.slice.call(arguments)));
};
fNOP.prototype = this.prototype;
fBound.prototype = new fNOP();
return fBound;
};
}
