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

Issue 2954903002: Media track ID visibility at BWE level

Created:
3 years, 5 months ago by alexnarest
Modified:
3 years, 2 months ago
CC:
webrtc-reviews_webrtc.org, interface-changes_webrtc.org, video-team_agora.io, yujie_mao (webrtc), zhengzhonghou_agora.io, stefan-webrtc, tlegrand-webrtc, tterriberry_mozilla.com, qiang.lu, niklas.enbom, peah-webrtc, the sun, mflodman
Target Ref:
refs/heads/master
Project:
webrtc
Visibility:
Public.

Description

Adds media track ID visibility at bitrate allocator level. Track IDs are assigned by application during track creation. Track IDs are used by custom bitrate allocation strategies to identify tracks. Track ID can be empty, in that case bitrate allocation strategies will not be able to handle these tracks specifically and will handle them as a default. BUG=webrtc:8243

Patch Set 1 : Media track ID visibility at BWE level #

Total comments: 4

Messages

Total messages: 19 (12 generated)
stefan-webrtc
To better understand how the trackid is going to be used it would be used ...
3 years, 5 months ago (2017-06-27 07:37:18 UTC) #3
alexnarest
This is a prequel to https://codereview.webrtc.org/2996643002/ PTAL
3 years, 3 months ago (2017-09-15 14:58:54 UTC) #10
Taylor Brandstetter
lgtm
3 years, 3 months ago (2017-09-16 00:47:48 UTC) #11
nisse-webrtc
lgtm In cl description, reference to "dependent cls" is not very helpful. Delete, and it ...
3 years, 3 months ago (2017-09-18 08:44:29 UTC) #12
alexnarest
Description updated. https://codereview.webrtc.org/2954903002/diff/60001/webrtc/call/audio_send_stream.h File webrtc/call/audio_send_stream.h (right): https://codereview.webrtc.org/2954903002/diff/60001/webrtc/call/audio_send_stream.h#newcode128 webrtc/call/audio_send_stream.h:128: std::string track_id; On 2017/09/18 08:44:28, nisse-webrtc wrote: ...
3 years, 2 months ago (2017-09-29 12:13:57 UTC) #17
stefan-webrtc
lgtm
3 years, 2 months ago (2017-09-29 12:19:34 UTC) #18
nisse-webrtc
3 years, 2 months ago (2017-09-29 12:32:47 UTC) #19
lgtm

Powered by Google App Engine
This is Rietveld 408576698