Merge pull request #297 from twpayne/fix-297

Canvas map is not immediately updated when layers are added or removed
openlayers · Mar 12, 2013 · f75d3ab · f75d3ab
2 parents 06f8d13 + 427c376
commit f75d3ab
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 31 deletions.
diff --git a/src/ol/renderer/dom/dommaprenderer.js b/src/ol/renderer/dom/dommaprenderer.js
@@ -47,15 +47,6 @@ ol.renderer.dom.Map = function(container, map) {
 goog.inherits(ol.renderer.dom.Map, ol.renderer.Map);
 
 
-/**
- * @inheritDoc
- */
-ol.renderer.dom.Map.prototype.addLayer = function(layer) {
-  goog.base(this, 'addLayer', layer);
-  this.getMap().render();
-};
-
-
 /**
  * @inheritDoc
  */

diff --git a/src/ol/renderer/maprenderer.js b/src/ol/renderer/maprenderer.js
@@ -79,6 +79,7 @@ goog.inherits(ol.renderer.Map, goog.Disposable);
 ol.renderer.Map.prototype.addLayer = function(layer) {
   var layerRenderer = this.createLayerRenderer(layer);
   this.setLayerRenderer(layer, layerRenderer);
+  this.getMap().render();
 };
 
 
@@ -224,6 +225,7 @@ ol.renderer.Map.prototype.handleLayersRemove = function(collectionEvent) {
  */
 ol.renderer.Map.prototype.removeLayer = function(layer) {
   goog.dispose(this.removeLayerRenderer(layer));
+  this.getMap().render();
 };
 
 

diff --git a/src/ol/renderer/webgl/webglmaprenderer.js b/src/ol/renderer/webgl/webglmaprenderer.js
@@ -216,17 +216,6 @@ ol.renderer.webgl.Map = function(container, map) {
 goog.inherits(ol.renderer.webgl.Map, ol.renderer.Map);
 
 
-/**
- * @inheritDoc
- */
-ol.renderer.webgl.Map.prototype.addLayer = function(layer) {
-  goog.base(this, 'addLayer', layer);
-  if (layer.getVisible()) {
-    this.getMap().render();
-  }
-};
-
-
 /**
  * @param {ol.Tile} tile Tile.
  * @param {number} magFilter Mag filter.
@@ -465,17 +454,6 @@ ol.renderer.webgl.Map.prototype.isTileTextureLoaded = function(tile) {
 };
 
 
-/**
- * @inheritDoc
- */
-ol.renderer.webgl.Map.prototype.removeLayer = function(layer) {
-  goog.base(this, 'removeLayer', layer);
-  if (layer.getVisible()) {
-    this.getMap().render();
-  }
-};
-
-
 /**
  * @inheritDoc
  */