Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(207)

Issue 3004783003: neteq_rtpplay: Add one more RTP header extension and fix some stats (Closed)

Created:
3 years, 3 months ago by hlundin-webrtc
Modified:
3 years, 3 months ago
Reviewers:
ivoc
CC:
webrtc-reviews_webrtc.org, AleBzk, peah-webrtc, tlegrand-webrtc, tterriberry_mozilla.com, audio-team_agora.io, hlundin-webrtc, kwiberg-webrtc, minyue-webrtc
Target Ref:
refs/heads/master
Project:
webrtc
Visibility:
Public.

Description

neteq_rtpplay: Add one more RTP header extension and fix some stats The extension ID for transport sequence number is added to the list of known RTP header extensions. Also, the minimum and maximum waiting time for packets is now aggregated as minimum and maximum, respectively, not as averages. BUG=none Review-Url: https://codereview.webrtc.org/3004783003 Cr-Commit-Position: refs/heads/master@{#19593} Committed: https://chromium.googlesource.com/external/webrtc/+/9657172a81f6caf150b431c2b756a0a2a69f850c

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -5 lines) Patch
M webrtc/modules/audio_coding/neteq/tools/neteq_rtpplay.cc View 4 chunks +12 lines, -5 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 8 (4 generated)
hlundin-webrtc
Ivo, PTAL. Small improvements to neteq_rtpplay.
3 years, 3 months ago (2017-08-29 15:09:39 UTC) #2
ivoc
lgtm!
3 years, 3 months ago (2017-08-29 15:18:23 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/3004783003/1
3 years, 3 months ago (2017-08-30 07:13:58 UTC) #5
commit-bot: I haz the power
3 years, 3 months ago (2017-08-30 07:41:41 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/9657172a81f6caf150b431c2b...

Powered by Google App Engine
This is Rietveld 408576698