Remove duplicate code for Audience population from snapshot

[WillieHabi]

Description

In protocol handler, we have duplicate for loops that populates Audience from Quorum members (lines 67-69 already do this). This PR deletes the duplicate code.

[msfluid-bot]

⯅ @fluid-example/bundle-size-tests: +11 Bytes

Metric Name	Baseline Size	Compare Size	Size Diff
aqueduct.js	459.57 KB	459.61 KB	⯅ +35 Bytes
azureClient.js	556.79 KB	556.76 KB	⯆ -29 Bytes
connectionState.js	724 Bytes	724 Bytes	■ No change
containerRuntime.js	259.19 KB	259.2 KB	⯅ +14 Bytes
fluidFramework.js	405.81 KB	405.82 KB	⯅ +14 Bytes
loader.js	134.24 KB	134.18 KB	⯆ -64 Bytes
map.js	42.46 KB	42.46 KB	⯅ +7 Bytes
matrix.js	148.29 KB	148.29 KB	⯅ +7 Bytes
odspClient.js	523.75 KB	523.73 KB	⯆ -29 Bytes
odspDriver.js	97.84 KB	97.86 KB	⯅ +21 Bytes
odspPrefetchSnapshot.js	42.81 KB	42.82 KB	⯅ +14 Bytes
sharedString.js	164.48 KB	164.49 KB	⯅ +7 Bytes
sharedTree.js	396.27 KB	396.28 KB	⯅ +7 Bytes
Total Size	3.31 MB	3.31 MB	⯅ +11 Bytes

---

Baseline commit: fcd15e0

Generated by 🚫 dangerJS against d480275

[ChumpChief]

Agree this looks duplicate and safe to remove, but then again it was duplicated explicitly in #20766 - maybe check with @vladsud if there's something we're missing here, e.g. if there's some weird interaction with external ProtocolOpHandlers?

[jason-ha]

Agree this looks duplicate and safe to remove, but then again it was duplicated explicitly in #20766 - maybe check with @vladsud if there's something we're missing here, e.g. if there's some weird interaction with external ProtocolOpHandlers?

I second double checking this. There was also a test added when the apparent duplicate code was added. I can't see why ordering would matter. Very curious.