Wrong setup when run in electron main process

[stephanbruny]

MQTT.js tries to use WebSockets when run in the main process of electron resulting in Error: 'WebSocket is not defined'.

The is-browser module tries to identify running in browser by checking webpack_require to be function, which it is, as the main process is bundled by WebPack.
Nevertheless, it is not a browser process.

I suggest changing or dropping the legacy-check in that module, or at least, drop the web pack check.

[stephanbruny]

Thank you very much @robertsLando !
That was quick

[robertsLando]

you welcome

[robertsLando]

I have a feel the change broke something: #1672 (comment)