Make take/drop not share state across subscriptions

[Jamesernator]

As spec-ed the behaviour of take/drop means this prints the values only once:

const obs = Observable.from([1,2,3,4,5,6]);
const firstThree = obs.take(3);

firstThree.forEach((v) => {
    console.log(v);
});

// This doesn't print anything
firstThree.forEach((v) => {
    console.log(v);
});

The intended behaviour I would expect is that each subscription has it's own counter.

The fix provided here just makes a new variable local to each subscription rather than re-using the argument to take/drop.

---

Preview | Diff

[w3cbot]

domfarolino marked as non substantive for IPR from ash-nazg.

[domfarolino]

Great catch! Looks like the tests already assert this as well, so this is a simple bug fix that we just land right now.