Use window.addEventListener instead of addEventListener

[cadorn]

Having addEventListener dangle as a global is not ideal.

Please use window.addEventListener to provide more context to static analysis tooling.

[calvinmetcalf]

do you have a specific issue that it is causing

[cadorn]

My static analysis tooling expects all access to global functions via window. I can fix that but it would be a hack on my side.

Having to inject only window as a global when executing this in a different context than browser makes things a lot cleaner than having to inject all the window globals in the to VM container as well.

Also, when looking at the code and you see just addEventListener you don't know what object you are registering events on so you have to see if addEventListener is declared elsewhere in the file before knowing its the window one.

[calvinmetcalf]

this the one?

[cadorn]

And this one:

catiline/src/nextTick.js

Line 24 in e4ca4fb

addEventListener('message', function (event) {