From 4701d117f7e574e498aab808c179bf31c9a03721 Mon Sep 17 00:00:00 2001 From: Tom Payne Date: Thu, 24 Jan 2013 23:37:20 +0100 Subject: [PATCH] Always dispatch change events on load/opacity/visible changes --- src/ol/renderer/dom/domlayerrenderer.js | 24 --------------------- src/ol/renderer/layerrenderer.js | 12 ++++++++--- src/ol/renderer/webgl/webgllayerrenderer.js | 24 --------------------- 3 files changed, 9 insertions(+), 51 deletions(-) diff --git a/src/ol/renderer/dom/domlayerrenderer.js b/src/ol/renderer/dom/domlayerrenderer.js index 261c24948f..4c78d42ed9 100644 --- a/src/ol/renderer/dom/domlayerrenderer.js +++ b/src/ol/renderer/dom/domlayerrenderer.js @@ -46,30 +46,6 @@ ol.renderer.dom.Layer.prototype.getTarget = function() { }; -/** - * @inheritDoc - */ -ol.renderer.dom.Layer.prototype.handleLayerLoad = function() { - this.getMap().render(); -}; - - -/** - * @inheritDoc - */ -ol.renderer.dom.Layer.prototype.handleLayerOpacityChange = function() { - this.getMap().render(); -}; - - -/** - * @inheritDoc - */ -ol.renderer.dom.Layer.prototype.handleLayerVisibleChange = function() { - this.getMap().render(); -}; - - /** * @param {ol.FrameState} frameState Frame state. * @param {ol.layer.LayerState} layerState Layer state. diff --git a/src/ol/renderer/layerrenderer.js b/src/ol/renderer/layerrenderer.js index b3243519e9..32be5e99fd 100644 --- a/src/ol/renderer/layerrenderer.js +++ b/src/ol/renderer/layerrenderer.js @@ -109,13 +109,17 @@ ol.renderer.Layer.prototype.handleLayerHueChange = goog.nullFunction; /** * @protected */ -ol.renderer.Layer.prototype.handleLayerLoad = goog.nullFunction; +ol.renderer.Layer.prototype.handleLayerLoad = function() { + this.dispatchChangeEvent(); +}; /** * @protected */ -ol.renderer.Layer.prototype.handleLayerOpacityChange = goog.nullFunction; +ol.renderer.Layer.prototype.handleLayerOpacityChange = function() { + this.dispatchChangeEvent(); +}; /** @@ -127,7 +131,9 @@ ol.renderer.Layer.prototype.handleLayerSaturationChange = goog.nullFunction; /** * @protected */ -ol.renderer.Layer.prototype.handleLayerVisibleChange = goog.nullFunction; +ol.renderer.Layer.prototype.handleLayerVisibleChange = function() { + this.dispatchChangeEvent(); +}; /** diff --git a/src/ol/renderer/webgl/webgllayerrenderer.js b/src/ol/renderer/webgl/webgllayerrenderer.js index 7fce8f5256..79195f1148 100644 --- a/src/ol/renderer/webgl/webgllayerrenderer.js +++ b/src/ol/renderer/webgl/webgllayerrenderer.js @@ -140,22 +140,6 @@ ol.renderer.webgl.Layer.prototype.handleLayerHueChange = function() { }; -/** - * @inheritDoc - */ -ol.renderer.webgl.Layer.prototype.handleLayerLoad = function() { - this.dispatchChangeEvent(); -}; - - -/** - * @inheritDoc - */ -ol.renderer.webgl.Layer.prototype.handleLayerOpacityChange = function() { - this.dispatchChangeEvent(); -}; - - /** * @inheritDoc */ @@ -167,14 +151,6 @@ ol.renderer.webgl.Layer.prototype.handleLayerSaturationChange = function() { }; -/** - * @inheritDoc - */ -ol.renderer.webgl.Layer.prototype.handleLayerVisibleChange = function() { - this.dispatchChangeEvent(); -}; - - /** * Handle webglcontextlost. */