CreateJS / PreloadJS

PreloadJS makes preloading assets & getting aggregate progress events easier in JavaScript. It uses XHR2 when available, and falls back to tag-based loading when not.

Home Page:http://createjs.com/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

RequestUtils.getTypeByExtension() and WebM format

djipco opened this issue · 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.