From 9251b830c3814bd585fc96a38a198debff2b8464 Mon Sep 17 00:00:00 2001 From: crschmidt Date: Fri, 19 Jan 2007 16:32:43 +0000 Subject: [PATCH] Approved commit for #473 by euzuro. Needs pullup to 2.3. git-svn-id: http://svn.openlayers.org/trunk/openlayers@2163 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf --- lib/OpenLayers/Tile/Image.js | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/lib/OpenLayers/Tile/Image.js b/lib/OpenLayers/Tile/Image.js index 113e5add5c..6539e933ed 100644 --- a/lib/OpenLayers/Tile/Image.js +++ b/lib/OpenLayers/Tile/Image.js @@ -117,12 +117,14 @@ OpenLayers.Tile.Image.prototype = this.imgDiv.className = 'olTileImage'; - /* checkImgURL *should* pretty predictably get called after the - createImage / createAlphaImageDiv onLoad handler */ + /* checkImgURL used to be used to called as a work around, but it + ended up hiding problems instead of solving them and broke things + like relative URLs. See discussion on the dev list: + http://openlayers.org/pipermail/dev/2007-January/000205.html OpenLayers.Event.observe( this.imgDiv, "load", this.checkImgURL.bindAsEventListener(this) ); - + */ this.layer.div.appendChild(this.imgDiv); if(this.layer.opacity != null) { @@ -139,6 +141,14 @@ OpenLayers.Tile.Image.prototype = * the imgDiv display to 'none', as either (a) it will be reset to visible * when the new URL loads in the image, or (b) we don't want to display * this tile after all because its new bounds are outside our maxExtent. + * + * This function should no longer be neccesary with the improvements to + * Grid.js in OpenLayers 2.3. The lack of a good isEquivilantURL function + * caused problems in 2.2, but it's possible that with the improved + * isEquivilant URL function, this might be neccesary at some point. + * + * See discussion in the thread at + * http://openlayers.org/pipermail/dev/2007-January/000205.html * * @private */