Update viewer on postUpdate rather than clock tick

CesiumGS · Sep 25, 2018 · b29cf7c · b29cf7c
1 parent 2014e68
commit b29cf7c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/Source/Scene/Scene.js b/Source/Scene/Scene.js
@@ -3286,6 +3286,10 @@ define([
         var us = context.uniformState;
         var frameState = this._frameState;
 
+        // Update with previous frame's time, assuming that render is called before picking.
+        this._preUpdate.raiseEvent(this, frameState.time);
+        this._postUpdate.raiseEvent(this, frameState.time);
+
         var view = this._defaultView;
         this._view = view;
 

diff --git a/Source/Widgets/Viewer/Viewer.js b/Source/Widgets/Viewer/Viewer.js
@@ -457,7 +457,7 @@ Either specify options.terrainProvider instead or set options.baseLayerPicker to
 
         var eventHelper = new EventHelper();
 
-        eventHelper.add(clock.onTick, Viewer.prototype._onTick, this);
+        eventHelper.add(scene.postUpdate, Viewer.prototype._onUpdate, this);
         eventHelper.add(scene.morphStart, Viewer.prototype._clearTrackedObject, this);
 
         // Selection Indicator
@@ -1586,9 +1586,7 @@ Either specify options.terrainProvider instead or set options.baseLayerPicker to
     /**
      * @private
      */
-    Viewer.prototype._onTick = function(clock) {
-        var time = clock.currentTime;
-
+    Viewer.prototype._onUpdate = function(scene, time) {
         var isUpdated = this._dataSourceDisplay.update(time);
         if (this._allowDataSourcesToSuspendAnimation) {
             this._clockViewModel.canAnimate = isUpdated;