shareReplay with refCount restarts with synchronous sources

[josepot]

Bug Report

Current Behavior
shareReplay with refCount restarts with synchronous sources

Reproduction

• REPL or Repo link:

I can't use stackblitz because since the latest release doesn't have #5521 then that bug masks the other one, but with the current code on master, this would happen:

const foo$ = from([1, 2, 3, 4, 5]).pipe(
  shareReplay({ refCount: true, bufferSize: 1})
)

foo$.subscribe(x => {
  console.log(x);
})
// 1
// 2
// 3
// 4
// 5

foo$.subscribe(x => {
  console.log(x);
})
// 1
// 2
// 3
// 4
// 5

Expected behavior
The second subscribe should only emit 5

Possible Solution
In this line also check that the subscription isn't closed:

if (subscription && !subscription.closed && useRefCount && refCount === 0) {

Additional context
See this comment from @leggechr and the comments afterwords.

[josepot]

I'm closing this issue because it was addressed by #5549