[OpenLayers-Trac] Re: [OpenLayers] #3419: Tile simplifications and partial rewrite

OpenLayers trac-20090302 at openlayers.org
Wed Aug 10 18:13:12 EDT 2011


#3419: Tile simplifications and partial rewrite
----------------------+-----------------------------------------------------
 Reporter:  ahocevar  |       Owner:  ahocevar    
     Type:  feature   |      Status:  new         
 Priority:  minor     |   Milestone:  2.12 Release
Component:  Tile      |     Version:  2.10        
 Keywords:            |       State:  Review      
----------------------+-----------------------------------------------------

Comment(by ahocevar):

 Hi Eric, I got the sequence in the if check right now, and indeed it makes
 much more sense now. I'm uploading a new patch, but the diff from the
 previous one is the following:
 {{{
 --- a/lib/OpenLayers/Tile/ImageMixin.js
 +++ b/lib/OpenLayers/Tile/ImageMixin.js
 @@ -84,10 +84,10 @@ OpenLayers.Tile.ImageMixin = {
          var data = this.backBufferData,
              ratio = data ? data.resolution / this.layer.getResolution() :
 1,
              noParent = this.frame.parentNode !== this.layer.div,
 -            needBuffer = ratio != 1 && this.backBufferMode == 1,
 +            needBuffer = ratio == 1 ? this.backBufferMode & 1 :
 this.backBuffe
              haveImage = this.frame && this.frame.childNodes.length > 0,
              backBufferUnavailable = !data || !data.frame &&
 this.isLoading;
 -        if (noParent || needBuffer || !haveImage ||
 backBufferUnavailable) {
 +        if (noParent || !needBuffer || !haveImage ||
 backBufferUnavailable) {
              this.setBackBufferData();
              return undefined;
          }
 }}}

-- 
Ticket URL: <http://trac.openlayers.org/ticket/3419#comment:23>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list