Don't clear newer packets from the video_coding::PacketBuffer when calling ClearTo;

Don't clear newer packets from the video_coding::PacketBuffer when calling ClearTo.

BUG=webrtc:8060
Review-Url: https://codereview.webrtc.org/2987013002
Cr-Commit-Position: refs/heads/master@{#19212}
Committed: https://chromium.googlesource.com/external/webrtc/+/c5fb4683e5841339ac1608be9dc853846038292a

[philipel, 2017-07-28 15:15:02]

Description was changed from

==========
Don't clear newer packets from the video_coding::PacketBuffer when calling
ClearTo;

BUG=None
==========

to

==========
Don't clear newer packets from the video_coding::PacketBuffer when calling
ClearTo.

BUG=None
==========

[philipel, 2017-07-28 15:15:02]

philipel@webrtc.org changed reviewers:
+ terelius@webrtc.org

[philipel, 2017-07-28 15:23:53]

philipel@webrtc.org changed reviewers:
+ sprang@webrtc.org

[sprang_webrtc, 2017-07-28 15:36:50]

Not that familiar with this code, but can rubber stamp if you need me to.
Also, please add a bug that this points to.

https://codereview.webrtc.org/2987013002/diff/1/webrtc/modules/video_coding/p...
File webrtc/modules/video_coding/packet_buffer.cc (right):

https://codereview.webrtc.org/2987013002/diff/1/webrtc/modules/video_coding/p...
webrtc/modules/video_coding/packet_buffer.cc:134: }
Is this case covered by the new test?

[philipel, 2017-08-01 11:22:03]

https://codereview.webrtc.org/2987013002/diff/1/webrtc/modules/video_coding/p...
File webrtc/modules/video_coding/packet_buffer.cc (right):

https://codereview.webrtc.org/2987013002/diff/1/webrtc/modules/video_coding/p...
webrtc/modules/video_coding/packet_buffer.cc:134: }
On 2017/07/28 15:36:50, sprang_webrtc wrote:
> Is this case covered by the new test?

It is covered by already existing tests.

[sprang_webrtc, 2017-08-02 10:56:37]

lgtm

[philipel, 2017-08-02 11:06:49]

Description was changed from

==========
Don't clear newer packets from the video_coding::PacketBuffer when calling
ClearTo.

BUG=None
==========

to

==========
Don't clear newer packets from the video_coding::PacketBuffer when calling
ClearTo.

BUG=webrtc:8060
==========

[philipel, 2017-08-02 11:06:59]

The CQ bit was checked by philipel@webrtc.org

[commit-bot: I haz the power, 2017-08-02 11:07:09]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-08-02 11:28:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1501672019315530, "parent_rev":
"c18f1d7c9432a2bf7d112820eb2b1dd2dbe2ba4b", "commit_rev":
"c5fb4683e5841339ac1608be9dc853846038292a"}

[commit-bot: I haz the power, 2017-08-02 11:29:02]

Description was changed from

==========
Don't clear newer packets from the video_coding::PacketBuffer when calling
ClearTo.

BUG=webrtc:8060
==========

to

==========
Don't clear newer packets from the video_coding::PacketBuffer when calling
ClearTo.

BUG=webrtc:8060

Review-Url: https://codereview.webrtc.org/2987013002
Cr-Commit-Position: refs/heads/master@{#19212}
Committed:
https://chromium.googlesource.com/external/webrtc/+/c5fb4683e5841339ac1608be9...
==========

[commit-bot: I haz the power, 2017-08-02 11:29:03]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/c5fb4683e5841339ac1608be9...