commit 97b6c757bc6f181e136cf35d90b23fc1872afaa9
author Erik Språng <sprang@webrtc.org> Thu Jul 25 09:16:34 2019 +0200
committer Commit Bot <commit-bot@chromium.org> Thu Jul 25 08:58:31 2019 +0000
tree 9238ea40e210e8ae1a7fa32871f2d1e04f20dd21
parent b79f564b53207549bff983e91276814b136d0615

New pacer: make FlexFEC and Video priority equal

Perf dashboard show a very minor change with the new pacer, for tests
that use flexfec. I have found that previously fec was in fact
prioritized at the same level as video, see eg PacketTypeToPriority()
in RTPSender.

With the new pacer we put fec in between video and padding.
Not sure if this is in fact an actual problem. In the non-loss case
the frame latency should actually be slighly lower, but on the other
hand if we have loss fec won't be applied until after the full frame
has been sent and so we may end up sending NACK before we apply the
FEC and recover a packet.

Just to avoid any problems let's revert to the old behavior.

Bug: webrtc:10633
Change-Id: I9a4210a64165a6e376c0c70ccaa07b0688cc58a5
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/146714
Reviewed-by: Sebastian Jansson <srte@webrtc.org>
Commit-Queue: Erik Språng <sprang@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28678}