From 5f798736e5e721ef21eecd31f58d78a942e9b2dd Mon Sep 17 00:00:00 2001 From: Danil Chapovalov Date: Thu, 9 Feb 2023 18:25:41 +0100 Subject: [PATCH] Delete stale TODOs related to VideoLayersAllocation extension No-Try: true Bug: webrtc:12000 Change-Id: I1ed3ece0eb000fe012ce5e26a6abaf640b422481 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/292880 Reviewed-by: Per Kjellander Commit-Queue: Danil Chapovalov Cr-Commit-Position: refs/heads/main@{#39291} --- .../rtp_rtcp/source/rtp_video_layers_allocation_extension.cc | 3 --- .../rtp_rtcp/source/rtp_video_layers_allocation_extension.h | 2 -- 2 files changed, 5 deletions(-) diff --git a/modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.cc b/modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.cc index 78fd7c9808..6b86ee553b 100644 --- a/modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.cc +++ b/modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.cc @@ -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; } diff --git a/modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.h b/modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.h index 3f1603bcd8..d59c922b36 100644 --- a/modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.h +++ b/modules/rtp_rtcp/source/rtp_video_layers_allocation_extension.h @@ -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;