From 6360e4497d5c86ecf76340c5f3ba29883bd7d546 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89ric=20Lemoine?= Date: Fri, 21 Nov 2014 15:35:59 +0100 Subject: [PATCH] Add a comment to WebGL ImageReplay A comment explaining why we don't need to call deleteProgram and deleteShader in the function returned by getDeleteResourcesFunction. --- src/ol/render/webgl/webglreplay.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/ol/render/webgl/webglreplay.js b/src/ol/render/webgl/webglreplay.js index ee9d2190f3..d78be4b5f9 100644 --- a/src/ol/render/webgl/webglreplay.js +++ b/src/ol/render/webgl/webglreplay.js @@ -201,6 +201,10 @@ ol.render.webgl.ImageReplay = function(tolerance, maxExtent) { */ ol.render.webgl.ImageReplay.prototype.getDeleteResourcesFunction = function(context) { + // We only delete our stuff here. The shaders and the program may + // be used by other ImageReplay instances (for other layers). And + // they will be deleted when disposing of the ol.webgl.Context + // object. goog.asserts.assert(!goog.isNull(this.verticesBuffer_)); goog.asserts.assert(!goog.isNull(this.indicesBuffer_)); var verticesBuffer = this.verticesBuffer_;