Library import pollute / override JS global URL namespace.

[midori0507]

If you use SSR (serverside rendering). window will be undefined.

Which this line

RecordRTC/RecordRTC.js

Line 1733 in 0f787fe

var URL = window.URL;

will set the var URL to undefined, cause window.URL is undefined.

After that:

This line

RecordRTC/RecordRTC.js

Line 1647 in 0f787fe

if (typeof URL === 'undefined') {

check and realize URL is undefined (cause you set it in the line above). And try to fallback by setting it with a mock-URL object
Later, global namespace is replace with this mocked URL.

Issues:

• This is really bad if you have SSR, you override global WHATWG URL with your mock URL object. Any call for example new URL() will show URL is not a constructor, cause you override it with your mock.

Suggestion:

• Move everything related to URL to use WHATWG URL, vanilla support from Nodejs v6 / 7. Will work on both client and server side.
• Dont mock and override global, not a good practice in js