CreateJS/PreloadJS

RequestUtils.getTypeByExtension() and WebM format

djipco opened this issue · 2 comments

In the RequestUtils.getTypeByExtension() method, the "webm" extension is there twice (line 110 and line 114). Is this intended?

I don't see how it could be detected as anything but createjs.Types.SOUND and I'm not sure this is necessarily the best default value for the WebM format.

I understand that we should specify the type explicitely but I stumbled upon that, while debugging a related issue, and figured I should report it.

It is probably just a mistake. If memory serves, there was a specific edge-case that that type handled. I am not sure it's still required.

Cheers,

I submitted PR #264 to fix this little mistake.