kripken/emscripten — pull request #2890 by caiiiycuk (2014-10-15 17:30:11+00:00)
Title: Fix copyIndexedColorData to work correct with opaqueFrontBuffer flag

Related to #2870

In past sdl implementation always make front buffer opaque on LockSurface, but now if opaqueFrontBuffer is set to false then we should set alpha value to 255 as in RGB color associated with the index.

@juj
I just deploy new version of openxcom (http://epicport.com/en/xcom) with EM_ASM("SDL.defaults.copyOnLock = false; SDL.defaults.discardOnLock = true; SDL.defaults.opaqueFrontBuffer = false;");. Looks like the problem place is in copyIndexedColorData function. This function take little performance advantages by this flag i think. Can you take a look on this function, it is possible to tune it performance? Also openxcom works blazing fast in chrome and much slower in firefox.