Emscripten closure compiler breaks WebGL 2 calls

[mosra]

There are some more cases following those I fixed back in emscripten-core/emscripten#3478 (for example glVertexAttribIPointer()). Temporary solution is to remove --closure 1 from toolchain flags (here).

• Submit a patch to Emscripten: One last fix to make Closure Compiler fully working with WebGL 2 emscripten-core/emscripten#7558
• Wait for it to be released: 1.38.21
• Wait for it to be common enough (as in, being available on Arch and on some Travis image without the need for brew update) and then re-add it to toolchains

[mosra]

I threw away the closure compiler for good, so this is no longer relevant.

💥

[mosra]

Reopened as the closure compiler can gain us quite nice file size savings. Need to fix the bugs tho.

[mosra]

Fix that makes all the test working with --closure 1 is submitted in emscripten-core/emscripten#7558.

[mosra]

Fix released in Emscripten 1.38.21, once the release gets common enough, I'll re-add the flag to all toolchains.

[mosra]

Since the flag causes significant compilation time increase, it won't get enabled by default. Instead, as of df6b414, there's a new section in the docs listing all extra Emscripten flags with code size / compilation time tradeoffs.