admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Delete stale TODOs related to VideoLayersAllocation extension

Browse Source 
No-Try: true
Bug: webrtc:12000
Change-Id: I1ed3ece0eb000fe012ce5e26a6abaf640b422481
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/292880
Reviewed-by: Per Kjellander <perkj@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#39291}
This commit is contained in:
Danil Chapovalov 2023-02-09 18:25:41 +01:00 committed by WebRTC LUCI CQ
parent 2b88704632
commit 5f798736e5
2 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.cc
View File

@ -58,9 +58,6 @@ bool AllocationIsValid(const VideoLayersAllocation& allocation) {
max_rtp_stream_idx = spatial_layer.rtp_stream_index;
}
if (allocation.resolution_and_frame_rate_is_valid) {
// TODO(danilchap): Add check width and height are no more than 0x10000
// when width and height become larger type and thus would support maximum
// resolution.
if (spatial_layer.width <= 0) {
return false;
}

2
modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.h
View File

@ -18,8 +18,6 @@
namespace webrtc {
// TODO(bugs.webrtc.org/12000): Note that this extensions is being developed and
// the wire format will likely change.
class RtpVideoLayersAllocationExtension {
public:
using value_type = VideoLayersAllocation;