Re-land (rebased for M50): Reset NACK/ACK list when parsing a new RTCP packet.

Re-land (rebased for M50): Reset NACK/ACK list when parsing a new RTCP packet.

The NACK/ACK frames/packets list need to be reset before parsing each RTCP packet as the parser is shared by all RTCP packets on sender side. This caused the performance regression and potentially video freezing as well.

TBR=xjz@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=600527
Review URL: https://codereview.chromium.org/1853253003

Cr-Commit-Position: refs/heads/master@{#385046}
(cherry picked from commit 411ea14c10f64b67a25985619583a5331e724d57)

[miu, 2016-04-07 21:36:49]

Description was changed from

==========
Re-land (rebased for M50): Reset NACK/ACK list when parsing a new RTCP packet.

The NACK/ACK frames/packets list need to be reset before parsing each RTCP
packet as the parser is shared by all RTCP packets on sender side. This caused
the performance regression and potentially video freezing as well.

BUG=600527

Review URL: https://codereview.chromium.org/1853253003

Cr-Commit-Position: refs/heads/master@{#385046}
(cherry picked from commit 411ea14c10f64b67a25985619583a5331e724d57)
==========

to

==========
Re-land (rebased for M50): Reset NACK/ACK list when parsing a new RTCP packet.

The NACK/ACK frames/packets list need to be reset before parsing each RTCP
packet as the parser is shared by all RTCP packets on sender side. This caused
the performance regression and potentially video freezing as well.

TBR=xjz@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=600527

Review URL: https://codereview.chromium.org/1853253003

Cr-Commit-Position: refs/heads/master@{#385046}
(cherry picked from commit 411ea14c10f64b67a25985619583a5331e724d57)
==========

[miu, 2016-04-07 21:36:49]

miu@chromium.org changed reviewers:
+ xjz@chromium.org

[miu, 2016-04-07 21:37:11]

xjz: FYI.  Attempt #2 to reland your fix.

[miu, 2016-04-07 21:37:16]

The CQ bit was checked by miu@chromium.org

[commit-bot: I haz the power, 2016-04-07 21:37:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1867883003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1867883003/1

[commit-bot: I haz the power, 2016-04-07 21:38:28]

Description was changed from

==========
Re-land (rebased for M50): Reset NACK/ACK list when parsing a new RTCP packet.

The NACK/ACK frames/packets list need to be reset before parsing each RTCP
packet as the parser is shared by all RTCP packets on sender side. This caused
the performance regression and potentially video freezing as well.

TBR=xjz@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=600527

Review URL: https://codereview.chromium.org/1853253003

Cr-Commit-Position: refs/heads/master@{#385046}
(cherry picked from commit 411ea14c10f64b67a25985619583a5331e724d57)
==========

to

==========
Re-land (rebased for M50): Reset NACK/ACK list when parsing a new RTCP packet.

The NACK/ACK frames/packets list need to be reset before parsing each RTCP
packet as the parser is shared by all RTCP packets on sender side. This caused
the performance regression and potentially video freezing as well.

TBR=xjz@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=600527

Review URL: https://codereview.chromium.org/1853253003

Cr-Commit-Position: refs/heads/master@{#385046}
(cherry picked from commit 411ea14c10f64b67a25985619583a5331e724d57)
==========

[commit-bot: I haz the power, 2016-04-07 21:38:29]

Committed patchset #1 (id:1)