Remove DOM layer in render, not in dispose

This commit is contained in:
Tom Payne
2013-04-18 19:16:44 +02:00
parent f7c0c26bed
commit 8831d25929
2 changed files with 9 additions and 9 deletions

View File

@@ -27,15 +27,6 @@ ol.renderer.dom.Layer = function(mapRenderer, layer, target) {
goog.inherits(ol.renderer.dom.Layer, ol.renderer.Layer);
/**
* @inheritDoc
*/
ol.renderer.dom.Layer.prototype.disposeInternal = function() {
goog.dom.removeNode(this.target);
goog.base(this, 'disposeInternal');
};
/**
* @protected
* @return {!Element} Target.

View File

@@ -4,6 +4,7 @@ goog.require('goog.array');
goog.require('goog.asserts');
goog.require('goog.dom');
goog.require('goog.dom.TagName');
goog.require('goog.object');
goog.require('goog.style');
goog.require('ol.css');
goog.require('ol.layer.ImageLayer');
@@ -86,6 +87,14 @@ ol.renderer.dom.Map.prototype.renderFrame = function(frameState) {
}
}, this);
goog.object.forEach(
this.getLayerRenderers(),
function(layerRenderer, layerKey) {
if (!(layerKey in frameState.layerStates)) {
goog.dom.removeNode(layerRenderer.getTarget());
}
});
if (!this.renderedVisible_) {
goog.style.showElement(this.layersPane_, true);
this.renderedVisible_ = true;