The RequestAnimationFrame.step and stepTimeout functions have been updated so that the new Frame is requested from raf before the main game step is called. This allows you to now stop the raf callback from within the game update or render loop. Fix phaserjs#3952

Author: photonstorm

CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Updates
 
 * `WebGLRenderer.deleteTexture` will check to see if the texture it is being asked to delete is the currently bound texture or not. If it is, it'll set the blank texture to be bound after deletion. This should stop `RENDER WARNING: there is no texture bound to the unit 0` errors if you destroy a Game Object, such as Text or TileSprite, from an async or timed process (thanks jamespierce)
+* The `RequestAnimationFrame.step` and `stepTimeout` functions have been updated so that the new Frame is requested from raf before the main game step is called. This allows you to now stop the raf callback from within the game update or render loop. Fix #3952 (thanks @tolimeh)
 
 ### Bug Fixes
 

src/dom/RequestAnimationFrame.js

@@ -94,14 +94,14 @@ var RequestAnimationFrame = new Class({
          */
         this.step = function step (timestamp)
         {
-            // DOMHighResTimeStamp
+            //  DOMHighResTimeStamp
             _this.lastTime = _this.tick;
 
             _this.tick = timestamp;
 
-            _this.callback(timestamp);
-
             _this.timeOutID = window.requestAnimationFrame(step);
+
+            _this.callback(timestamp);
         };
 
         /**
@@ -122,9 +122,9 @@ var RequestAnimationFrame = new Class({
 
             _this.tick = d;
 
-            _this.callback(d);
-
             _this.timeOutID = window.setTimeout(stepTimeout, delay);
+
+            _this.callback(d);
         };
     },