Use TileCache instead of custom structure

This commit is contained in:
Andreas Hocevar
2020-01-05 12:25:44 +01:00
parent ae1ee192f3
commit 2875685b3c
3 changed files with 10 additions and 28 deletions

View File

@@ -18,11 +18,6 @@ class VectorTile extends Tile {
super(tileCoord, state, opt_options); super(tileCoord, state, opt_options);
/**
* @type {number}
*/
this.consumers = 0;
/** /**
* Extent of this tile; set by the source. * Extent of this tile; set by the source.
* @type {import("./extent.js").Extent} * @type {import("./extent.js").Extent}

View File

@@ -13,6 +13,7 @@ import {buffer as bufferExtent, getIntersection, intersects} from '../extent.js'
import EventType from '../events/EventType.js'; import EventType from '../events/EventType.js';
import {loadFeaturesXhr} from '../featureloader.js'; import {loadFeaturesXhr} from '../featureloader.js';
import {equals} from '../array.js'; import {equals} from '../array.js';
import TileCache from '../TileCache.js';
/** /**
* @typedef {Object} Options * @typedef {Object} Options
@@ -139,9 +140,9 @@ class VectorTile extends UrlTile {
/** /**
* @private * @private
* @type {Object<string, import("../VectorTile.js").default>} * @type {TileCache}
*/ */
this.sourceTileByKey_ = {}; this.sourceTileCache = new TileCache(this.tileCache.highWaterMark);
/** /**
* @private * @private
@@ -227,7 +228,7 @@ class VectorTile extends UrlTile {
*/ */
clear() { clear() {
this.tileCache.clear(); this.tileCache.clear();
this.sourceTileByKey_ = {}; this.sourceTileCache.clear();
this.sourceTilesByTileKey_ = {}; this.sourceTilesByTileKey_ = {};
} }
@@ -269,8 +270,8 @@ class VectorTile extends UrlTile {
const tileUrl = this.tileUrlFunction(sourceTileCoord, pixelRatio, projection); const tileUrl = this.tileUrlFunction(sourceTileCoord, pixelRatio, projection);
let sourceTile; let sourceTile;
if (tileUrl !== undefined) { if (tileUrl !== undefined) {
if (tileUrl in this.sourceTileByKey_) { if (this.sourceTileCache.containsKey(tileUrl)) {
sourceTile = this.sourceTileByKey_[tileUrl]; sourceTile = this.sourceTileCache.get(tileUrl);
const state = sourceTile.getState(); const state = sourceTile.getState();
if (state === TileState.LOADED || state === TileState.ERROR || state === TileState.EMPTY) { if (state === TileState.LOADED || state === TileState.ERROR || state === TileState.EMPTY) {
sourceTiles.push(sourceTile); sourceTiles.push(sourceTile);
@@ -282,7 +283,7 @@ class VectorTile extends UrlTile {
sourceTile.extent = sourceTileGrid.getTileCoordExtent(sourceTileCoord); sourceTile.extent = sourceTileGrid.getTileCoordExtent(sourceTileCoord);
sourceTile.projection = projection; sourceTile.projection = projection;
sourceTile.resolution = sourceTileGrid.getResolution(sourceTileCoord[0]); sourceTile.resolution = sourceTileGrid.getResolution(sourceTileCoord[0]);
this.sourceTileByKey_[tileUrl] = sourceTile; this.sourceTileCache.set(tileUrl, sourceTile);
sourceTile.addEventListener(EventType.CHANGE, this.handleTileChange.bind(this)); sourceTile.addEventListener(EventType.CHANGE, this.handleTileChange.bind(this));
sourceTile.load(); sourceTile.load();
} }
@@ -333,6 +334,7 @@ class VectorTile extends UrlTile {
this.addSourceTiles(tile, sourceTiles); this.addSourceTiles(tile, sourceTiles);
} }
} }
this.sourceTileCache.expireCache({});
return sourceTiles; return sourceTiles;
} }
@@ -342,28 +344,13 @@ class VectorTile extends UrlTile {
*/ */
addSourceTiles(tile, sourceTiles) { addSourceTiles(tile, sourceTiles) {
this.sourceTilesByTileKey_[tile.getKey()] = sourceTiles; this.sourceTilesByTileKey_[tile.getKey()] = sourceTiles;
for (let i = 0, ii = sourceTiles.length; i < ii; ++i) {
sourceTiles[i].consumers++;
}
} }
/** /**
* @param {VectorRenderTile} tile Tile. * @param {VectorRenderTile} tile Tile.
*/ */
removeSourceTiles(tile) { removeSourceTiles(tile) {
const tileKey = tile.getKey(); delete this.sourceTilesByTileKey_[tile.getKey()];
if (tileKey in this.sourceTilesByTileKey_) {
const sourceTiles = this.sourceTilesByTileKey_[tileKey];
for (let i = 0, ii = sourceTiles.length; i < ii; ++i) {
const sourceTile = sourceTiles[i];
sourceTile.consumers--;
if (sourceTile.consumers === 0) {
sourceTile.dispose();
delete this.sourceTileByKey_[sourceTile.getKey()];
}
}
}
delete this.sourceTilesByTileKey_[tileKey];
} }
/** /**

View File

@@ -315,7 +315,7 @@ describe('ol.renderer.canvas.VectorTileLayer', function() {
tileClass: TileClass, tileClass: TileClass,
tileGrid: createXYZ() tileGrid: createXYZ()
}); });
source.sourceTileByKey_[sourceTile.getKey()] = sourceTile; source.sourceTileCache.set('0/0/0.mvt', sourceTile);
source.sourceTilesByTileKey_[sourceTile.getKey()] = [sourceTile]; source.sourceTilesByTileKey_[sourceTile.getKey()] = [sourceTile];
executorGroup = {}; executorGroup = {};
source.getTile = function() { source.getTile = function() {