Remove event listener with goog.events.unlistenByKey

This commit is contained in:
Frederic Junod
2013-01-28 10:26:09 +01:00
parent f208e99c64
commit 01098c4d8c
+8 -4
View File
@@ -117,6 +117,12 @@ ol.MapBrowserEventHandler = function(map) {
*/ */
this.timestamp_ = 0; this.timestamp_ = 0;
/**
* @type {?number}
* @private
*/
this.clickListenerKey_ = null;
/** /**
* @type {Array.<number>} * @type {Array.<number>}
* @private * @private
@@ -131,7 +137,7 @@ ol.MapBrowserEventHandler = function(map) {
var element = this.map_.getViewport(); var element = this.map_.getViewport();
if (!ol.BrowserFeature.HAS_TOUCH) { if (!ol.BrowserFeature.HAS_TOUCH) {
goog.events.listen(element, this.clickListenerKey_ = goog.events.listen(element,
[goog.events.EventType.CLICK, goog.events.EventType.DBLCLICK], [goog.events.EventType.CLICK, goog.events.EventType.DBLCLICK],
this.click_, false, this); this.click_, false, this);
} }
@@ -286,9 +292,7 @@ ol.MapBrowserEventHandler.prototype.disposeInternal = function() {
goog.events.EventType.TOUCHEND : goog.events.EventType.TOUCHEND :
goog.events.EventType.MOUSEUP, goog.events.EventType.MOUSEUP,
this.handleUp_, false, this); this.handleUp_, false, this);
goog.events.unlisten(element, goog.events.unlistenByKey(this.clickListenerKey_);
[goog.events.EventType.CLICK, goog.events.EventType.DBLCLICK],
this.click_, false, this);
if (!goog.isNull(this.dragListenerKeys_)) { if (!goog.isNull(this.dragListenerKeys_)) {
goog.array.forEach(this.dragListenerKeys_, goog.events.unlistenByKey); goog.array.forEach(this.dragListenerKeys_, goog.events.unlistenByKey);
this.dragListenerKeys_ = null; this.dragListenerKeys_ = null;