Queue size performance issue

[matthiasbayer]

Hi,

thanks for maintaining this repo, great work!

Are you aware of the performance issues mentioned here:

• Chromium Bug: https://bugs.chromium.org/p/chromium/issues/detail?id=681493
• Explaination: https://groups.google.com/forum/#!msg/v8-users/Hj78A4yhQS4/edz6-NsyCAAJ
• Issue on sd-streams repo: Possible scaling issue with QueueContainer<V> stardazed/sd-streams#1

I created a CodePen using the code from the bug report:

• Native: https://codepen.io/anon/pen/gEpXXb
  

• Polyfill: https://codepen.io/anon/pen/EMEoaa
  

I'd like to use this polyfill in production, but the performance drop, even if unlikely to happen, is a very bad experience for the user.

Do you think it's possible to include a fix (e.g. something like stardazed/sd-streams@1128f0d) into this repository? Or is this not possible because it would be non-standard?

Looking forward to hear you opinion on this.

[MattiasBuelens]

Ouch, that's quite the performance hit. Thanks for reporting!

I'll look into implementing a fix similar to the one for sd-streams. I'm okay with diverting from the reference implementation, as long as we keep passing all web platform tests. 🙂