Handle expiring subtrees

Author: lilleyse

Source/Scene/Cesium3DTile.js

@@ -120,8 +120,6 @@ define([
          */
         this.computedTransform = computedTransform;
 
-        this._transformDirty = true;
-
         this._boundingVolume = this.createBoundingVolume(header.boundingVolume, computedTransform);
         this._boundingVolume2D = undefined;
 
@@ -422,20 +420,6 @@ define([
             }
         },
 
-        /**
-         * Whether the computedTransform has changed this frame.
-         *
-         * @memberof Cesium3DTile.prototype
-         *
-         * @type {Boolean}
-         * @readonly
-         */
-        transformDirty : {
-            get : function() {
-                return this._transformDirty;
-            }
-        },
-
         /**
          * @readonly
          * @private
@@ -550,15 +534,20 @@ define([
 
     var scratchJulianDate = new JulianDate();
 
-    function updateExpiration(tile) {
-        if (defined(tile.expireDate) && tile.contentReady && !tile.hasEmptyContent) {
+    /**
+     * Update whether the tile has expired.
+     *
+     * @private
+     */
+    Cesium3DTile.prototype.updateExpiration = function() {
+        if (defined(this.expireDate) && this.contentReady && !this.hasEmptyContent) {
             var now = JulianDate.now(scratchJulianDate);
-            if (JulianDate.lessThan(tile.expireDate, now)) {
-                tile._contentState = Cesium3DTileContentState.EXPIRED;
-                tile._expiredContent = tile._content;
+            if (JulianDate.lessThan(this.expireDate, now)) {
+                this._contentState = Cesium3DTileContentState.EXPIRED;
+                this._expiredContent = this._content;
             }
         }
-    }
+    };
 
     function updateExpireDate(tile) {
         if (defined(tile.expireDuration)) {
@@ -620,9 +609,9 @@ define([
 
         promise.then(function(arrayBuffer) {
             if (that.isDestroyed()) {
-                return when.reject('tileset is destroyed');
+                // Tile is unloaded before the content finishes loading
+                return when.reject('tile is destroyed');
             }
-
             var uint8Array = new Uint8Array(arrayBuffer);
             var magic = getMagic(uint8Array);
             var contentFactory = Cesium3DTileContentFactory[magic];
@@ -642,6 +631,11 @@ define([
             that._contentReadyToProcessPromise.resolve(content);
 
             content.readyPromise.then(function(content) {
+                if (that.isDestroyed()) {
+                    // Tile is unloaded before the content finishes processing
+                    that._content.destroy();
+                    return when.reject('tile is destroyed');
+                }
                 updateExpireDate(that);
                 that._contentState = Cesium3DTileContentState.READY;
                 that._contentReadyPromise.resolve(content);
@@ -883,9 +877,8 @@ define([
     Cesium3DTile.prototype.updateTransform = function(parentTransform) {
         parentTransform = defaultValue(parentTransform, Matrix4.IDENTITY);
         var computedTransform = Matrix4.multiply(parentTransform, this.transform, scratchTransform);
-        var transformDirty = !Matrix4.equals(computedTransform, this.computedTransform);
-        if (transformDirty) {
-            this._transformDirty = true;
+        var transformChanged = !Matrix4.equals(computedTransform, this.computedTransform);
+        if (transformChanged) {
             Matrix4.clone(computedTransform, this.computedTransform);
 
             // Update the bounding volumes
@@ -969,9 +962,7 @@ define([
      */
     Cesium3DTile.prototype.update = function(tileset, frameState) {
         applyDebugSettings(this, tileset, frameState);
-        updateExpiration(this);
         updateContent(this, tileset, frameState);
-        this._transformDirty = false;
     };
 
     var scratchCommandList = [];
@@ -1006,10 +997,11 @@ define([
      */
     Cesium3DTile.prototype.destroy = function() {
         this._content = this._content && this._content.destroy();
-        this._expiredContent = this._expiredContent && this._expiredContent.destroy();
+        this._expiredContent = this._expiredContent && !this._expiredContent.isDestroyed() && this._expiredContent.destroy();
         this._debugBoundingVolume = this._debugBoundingVolume && this._debugBoundingVolume.destroy();
         this._debugContentBoundingVolume = this._debugContentBoundingVolume && this._debugContentBoundingVolume.destroy();
         this._debugViewerRequestVolume = this._debugViewerRequestVolume && this._debugViewerRequestVolume.destroy();
+
         return destroyObject(this);
     };
 

Source/Scene/Cesium3DTileset.js

@@ -1255,18 +1255,13 @@ define([
         }
 
         var stats = tileset._statistics;
-        var expired = tile.contentExpired;
         var requested = tile.requestContent();
 
         if (!requested) {
             ++stats.numberOfAttemptedRequests;
             return;
         }
 
-        if (expired && tile.hasTilesetContent) {
-            unloadSubtree(tileset, tile);
-        }
-
         ++stats.numberOfPendingRequests;
 
         var removeFunction = removeFromProcessingQueue(tileset, tile);
@@ -1591,6 +1586,7 @@ define([
         tile._sse = getScreenSpaceError(tileset, tile.geometricError, tile, frameState);
         tile.selected = false;
         tile._finalResolution = false;
+        tile.updateExpiration();
         touch(tileset, tile, outOfCore);
     }
 
@@ -1623,7 +1619,12 @@ define([
         var loadSiblings = tileset.loadSiblings;
 
         if (tile.hasTilesetContent) {
-            updateAndPushChildren(tileset, tile, frameState, stack, loadSiblings, outOfCore);
+            if (tile.contentExpired) {
+                loadTile(tile);
+                unloadSubtree(tileset, tile);
+            } else {
+                updateAndPushChildren(tileset, tile, frameState, stack, loadSiblings, outOfCore);
+            }
         } else {
             if (tile.refine === Cesium3DTileRefine.ADD) {
                 loadAndAddToQueue(tileset, tile, finalQueue);
@@ -1982,7 +1983,7 @@ define([
     }
 
     function unloadTile(tileset, tile) {
-        if (!tile.hasRenderableContent) {
+        if (!tile.hasRenderableContent || !tile.contentReady) {
             return;
         }
 
@@ -1997,7 +1998,7 @@ define([
         tile.unloadContent();
     }
 
-    function unloadTiles(tileset, frameState) {
+    function unloadTiles(tileset) {
         var trimTiles = tileset._trimTiles;
         tileset._trimTiles = false;
 
@@ -2100,7 +2101,7 @@ define([
         updateTiles(this, frameState);
 
         if (outOfCore) {
-            unloadTiles(this, frameState);
+            unloadTiles(this);
         }
 
         // Events are raised (added to the afterRender queue) here since promises

Source/Scene/PointCloud3DTileContent.js

@@ -115,6 +115,7 @@ define([
         this._quantizedVolumeScale = undefined;
         this._quantizedVolumeOffset = undefined;
 
+        this._modelMatrix = Matrix4.clone(Matrix4.IDENTITY);
         this._mode = undefined;
 
         /**
@@ -1138,7 +1139,8 @@ define([
      * Part of the {@link Cesium3DTileContent} interface.
      */
     PointCloud3DTileContent.prototype.update = function(tileset, frameState) {
-        var updateModelMatrix = this._tile.transformDirty || this._mode !== frameState.mode;
+        var modelMatrixChanged = !Matrix4.equals(this._modelMatrix, this._tile.computedTransform);
+        var updateModelMatrix = modelMatrixChanged || this._mode !== frameState.mode;
         this._mode = frameState.mode;
 
         if (!defined(this._drawCommand)) {