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

Temporary add 'RTPVideoHeaderH264::nalus_length'.

Browse Source 
This is a forward fix for https://webrtc-review.googlesource.com/c/src/+/354622 that breaks client code using nalus_length.

No-Try: true
Change-Id: Ic0fc41696e408adefe4eb8792150a64b1eab49da
Bug: None
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/354840
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Owners-Override: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#42493}
This commit is contained in:
Jeremy Leconte 2024-06-17 09:07:27 +02:00 committed by WebRTC LUCI CQ
parent 72302cc5e4
commit 04dd95fcac
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/rtp_rtcp/source/video_rtp_depacketizer_h264.cc
View File

@ -221,6 +221,7 @@ absl::optional<VideoRtpDepacketizer::ParsedRtpPayload> ProcessStapAOrSingleNalu(
}
h264_header.nalus.push_back(nalu);
++h264_header.nalus_length;
}
return parsed_payload;
@ -279,6 +280,7 @@ absl::optional<VideoRtpDepacketizer::ParsedRtpPayload> ParseFuaNalu(
h264_header.nalu_type = original_nal_type;
if (first_fragment) {
h264_header.nalus = {nalu};
h264_header.nalus_length = 1;
}
return parsed_payload;
}

2
modules/video_coding/codecs/h264/include/h264_globals.h
View File

@ -83,6 +83,8 @@ struct RTPVideoHeaderH264 {
// The packetization type of this buffer - single, aggregated or fragmented.
H264PacketizationTypes packetization_type;
std::vector<NaluInfo> nalus;
// TODO - jleconte: Remove 'nalus_length' once all the code has been migrated.
size_t nalus_length;
// The packetization mode of this transport. Packetization mode
// determines which packetization types are allowed when packetizing.
H264PacketizationMode packetization_mode;