New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 5 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
NextAction: ----
OS: ----
Pri: 1
Type: Enhancement



Sign in to add a comment

Log buffer flush events in NetEq

Project Member Reported by henrik.lundin@webrtc.org, Oct 17 2014

Issue description

Log when packet_buffer_->InsertPacketList(...) returns PacketBuffer::kFlushed in NetEqImpl::InsertPacketInternal(...).

 
Project Member

Comment 1 by tina.legrand@webrtc.org, Oct 30 2014

Labels: EngTriaged IceBox
Project Member

Comment 2 by tnakamura@webrtc.org, Nov 4 2015

This bug hasn't been modified for more than a year. Is this still a valid open issue?
Project Member

Comment 3 by henrik.lundin@webrtc.org, Nov 5 2015

We write to the log when the buffer is flooded and flushed, but nothing is propagated outside of NetEq. Might still be a good idea to do this.
Project Member

Comment 4 by sheriffbot@chromium.org, Dec 12 2016

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been available for more than 365 days, and should be re-evaluated. Please re-triage this issue.
The Hotlist-Recharge-Cold label is applied for tracking purposes, and should not be removed after re-triaging the issue.


Project Member

Comment 5 by henrik.lundin@webrtc.org, Dec 13 2016

Cc: -henrik.lundin@webrtc.org tina.legrand@webrtc.org
Labels: Pri-3
Owner: henrik.lundin@webrtc.org
Status: Assigned (was: Untriaged)
Summary: Log buffer flush events in NetEq (was: Log buffer flush events to NetEqNetworkStatistics struct and surface to libjingle)
We still want to do this, but maybe not to the NetEqNetworkStatistics struct. UMA may be more suitable.
Project Member

Comment 6 by anatolid@webrtc.org, Dec 14 2016

Labels: -IceBox
Project Member

Comment 7 by tina.legrand@webrtc.org, Feb 7 2018

Henrik, should we bump up the priority of this?
Project Member

Comment 8 by henrik.lundin@webrtc.org, Feb 8 2018

Labels: -Pri-3 Pri-1
Yes, we should.

Sign in to add a comment