[MM-42737] Use stream ids instead of track ids #38
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
After some debugging (thanks @cpoile 🙌 ) and knowledge gathering I figured out that relying on local track ids was a dangerous move as they are not passed consistently. Stream IDs instead seem to be the way to go, so going with those this time around.
This solution does also remove the annoying browser specific check. The moral of the story is: I should have listened to Firefox when it told me something was clearly wrong with my intended logic.
Oh, and this is mostly why R&D call failed so emphatically.
Ticket Link
https://mattermost.atlassian.net/browse/MM-42737