fix(rxjs): no longer requires dom lib

[benlesh]

Adds interfaces to cover what is used from DOM in fromEvent so that types are carried through, but dom lib isn not required by default even if you are not using fromEvent

fixes: #3558

[imhoffd]

You can use isArrayLike here, right? https://github.com/ReactiveX/rxjs/blob/master/src/internal/util/isArrayLike.ts

[kherock]

Hey, this has some major issues when you want to do fromEvent(window, 'message'). window isn't a NodeList or HTMLCollection but it does have a .length property which refers to the number of frames in the window. As-is, this will attempt to add event listeners to all sub-frames on the window and cause an error of the form

Uncaught DOMException: Blocked a frame with origin "..." from accessing a cross-origin frame.

when subscribing.

[cartant]

@specialkk Thanks for pointing out a problem.

Could you please create an issue and, from it, reference this PR?

I'm not a maintainer of this particular repo, but I think I speak for all maintainers in saying that it's preferable to have new issues opened, than to have comments made on closed issues or merged PRs.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.