If you pass zero as the width or height when creating a TileSprite it will now use the dimensions of the texture frame as the size of the TileSprite. Fix phaserjs#4073

Author: photonstorm

CHANGELOG.md

@@ -10,6 +10,7 @@
 
 * `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)
+* If you pass zero as the width or height when creating a TileSprite it will now use the dimensions of the texture frame as the size of the TileSprite. Fix #4073 (thanks @jcyuan)
 
 ### Bug Fixes
 

src/gameobjects/tilesprite/TileSprite.js

@@ -64,8 +64,8 @@ var _FLAG = 8; // 1000
  * @param {Phaser.Scene} scene - The Scene to which this Game Object belongs. A Game Object can only belong to one Scene at a time.
  * @param {number} x - The horizontal position of this Game Object in the world.
  * @param {number} y - The vertical position of this Game Object in the world.
- * @param {number} width - The width of the Game Object.
- * @param {number} height - The height of the Game Object.
+ * @param {integer} width - The width of the Game Object. If zero it will use the size of the texture frame.
+ * @param {integer} height - The height of the Game Object. If zero it will use the size of the texture frame.
  * @param {string} textureKey - The key of the Texture this Game Object will use to render with, as stored in the Texture Manager.
  * @param {(string|integer)} [frameKey] - An optional frame from the Texture this Game Object is rendering with.
  */
@@ -96,13 +96,24 @@ var TileSprite = new Class({
 
     function TileSprite (scene, x, y, width, height, textureKey, frameKey)
     {
-        width = Math.floor(width);
-        height = Math.floor(height);
-
         var renderer = scene.sys.game.renderer;
 
         GameObject.call(this, scene, 'TileSprite');
 
+        var displayTexture = scene.sys.textures.get(textureKey);
+        var displayFrame = displayTexture.get(frameKey);
+
+        if (!width || !height)
+        {
+            width = displayFrame.width;
+            height = displayFrame.height;
+        }
+        else
+        {
+            width = Math.floor(width);
+            height = Math.floor(height);
+        }
+
         /**
          * Internal tile position vector.
          *
@@ -172,7 +183,7 @@ var TileSprite = new Class({
          * @private
          * @since 3.12.0
          */
-        this.displayTexture = scene.sys.textures.get(textureKey);
+        this.displayTexture = displayTexture;
 
         /**
          * The Frame the TileSprite is using as its fill pattern.
@@ -182,7 +193,7 @@ var TileSprite = new Class({
          * @private
          * @since 3.12.0
          */
-        this.displayFrame = this.displayTexture.get(frameKey);
+        this.displayFrame = displayFrame;
 
         /**
          * The internal crop data object, as used by `setCrop` and passed to the `Frame.setCropUVs` method.
@@ -219,7 +230,7 @@ var TileSprite = new Class({
          * @type {integer}
          * @since 3.0.0
          */
-        this.potWidth = GetPowerOfTwo(this.displayFrame.width);
+        this.potWidth = GetPowerOfTwo(displayFrame.width);
 
         /**
          * The next power of two value from the height of the Fill Pattern frame.
@@ -228,7 +239,7 @@ var TileSprite = new Class({
          * @type {integer}
          * @since 3.0.0
          */
-        this.potHeight = GetPowerOfTwo(this.displayFrame.height);
+        this.potHeight = GetPowerOfTwo(displayFrame.height);
 
         /**
          * The Canvas that the TileSprites texture is rendered to.

src/gameobjects/tilesprite/TileSpriteCreator.js

@@ -15,8 +15,8 @@ var TileSprite = require('./TileSprite');
  *
  * @property {number} [x=0] - The x coordinate of the Tile Sprite.
  * @property {number} [y=0] - The y coordinate of the Tile Sprite.
- * @property {number} [width=512] - The width of the Tile Sprite.
- * @property {number} [height=512] - The height of the Tile Sprite.
+ * @property {integer} [width=512] - The width of the Tile Sprite. If zero it will use the size of the texture frame.
+ * @property {integer} [height=512] - The height of the Tile Sprite. If zero it will use the size of the texture frame.
  * @property {string} [key=''] - The key of the Texture this Tile Sprite will use to render with, as stored in the Texture Manager.
  * @property {string} [frame=''] - An optional frame from the Texture this Tile Sprite is rendering with.
  */

src/gameobjects/tilesprite/TileSpriteFactory.js

@@ -17,8 +17,8 @@ var GameObjectFactory = require('../GameObjectFactory');
  *
  * @param {number} x - The horizontal position of this Game Object in the world.
  * @param {number} y - The vertical position of this Game Object in the world.
- * @param {number} width - The width of the Game Object.
- * @param {number} height - The height of the Game Object.
+ * @param {integer} width - The width of the Game Object. If zero it will use the size of the texture frame.
+ * @param {integer} height - The height of the Game Object. If zero it will use the size of the texture frame.
  * @param {string} texture - The key of the Texture this Game Object will use to render with, as stored in the Texture Manager.
  * @param {(string|integer)} [frame] - An optional frame from the Texture this Game Object is rendering with.
  *