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 */