Don't create new canvases for TMX Tile Objects.

[parasyte]

While debugging some changes in the WebGL Renderer, I found that the Tile Objects used in the platformer (the spikes) each have their own canvas object. This makes caching difficult, and will break the batching logic (#591), because none of them share a texture.

The code that creates the canvas is me.TMXTileset.getTileImage:

melonJS/src/level/TMXTileset.js

Line 302 in 70caf9c

getTileImage : function (tmxTile) {

It is called by the TMXObjectGroup class, when the object has a gid property:

melonJS/src/level/TMXObjectGroup.js

Line 261 in 07c1c83

this.setImage(this.gid, tilesets);

[obiot]

what should we create then ? and entity with a body and a renderable component ?

[parasyte]

It would need the Texture Atlas work that I mentioned in #591, but then this one could just change from creating a new canvas to instead referencing the original texture and an atlas index.

[parasyte]

#591 depends on this bug getting fixed.

Uncaught me.video.Error: Texture cache overflow: 16 texture units available.

Throws the exception after reloading the maps in the platformer example a few times.

[parasyte]

This one was fixed by #592