commit 4275448cbc6074f61d4ea1bf610b02adb1e77dae
author philipel <philipel@webrtc.org> Fri Sep 24 10:26:53 2021
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Sep 24 11:18:42 2021
tree 797f08ff985818c4db07b99a9c0d18b2fdaddf29
parent fdea8e6244fe33a95fd35bcb19fe39cd4e86abf9

Use new CopyOnWriteBuffer ctor/append function in H264PacketBuffer.

Bug: none
Change-Id: I02aace124f3f5a578a52348675b346ff890d8d82
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/232900
Commit-Queue: Philip Eliasson <philipel@webrtc.org>
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35082}

modules/video_coding/h264_packet_buffer.cc

diff --git a/modules/video_coding/h264_packet_buffer.cc b/modules/video_coding/h264_packet_buffer.cc
index 8968539..7b821a1 100644
--- a/modules/video_coding/h264_packet_buffer.cc
+++ b/modules/video_coding/h264_packet_buffer.cc
@@ -107,13 +107,13 @@
       if (IsFirstPacketOfFragment(h264_header)) {
         result.AppendData(kStartCode);
       }
-      result.AppendData(payload.data(), payload.size());
+      result.AppendData(payload);
       return result;
     }
 
     case kH264SingleNalu: {
       result.AppendData(kStartCode);
-      result.AppendData(payload.data(), payload.size());
+      result.AppendData(payload);
       return result;
     }
   }

modules/video_coding/h264_packet_buffer_unittest.cc

diff --git a/modules/video_coding/h264_packet_buffer_unittest.cc b/modules/video_coding/h264_packet_buffer_unittest.cc
index 3d2a432..4f2331d 100644
--- a/modules/video_coding/h264_packet_buffer_unittest.cc
+++ b/modules/video_coding/h264_packet_buffer_unittest.cc
@@ -198,15 +198,14 @@
 }
 
 rtc::CopyOnWriteBuffer Packet::BuildFuaPayload() const {
-  return rtc::CopyOnWriteBuffer(nalu_payloads_[0].data(),
-                                nalu_payloads_[0].size());
+  return rtc::CopyOnWriteBuffer(nalu_payloads_[0]);
 }
 
 rtc::CopyOnWriteBuffer Packet::BuildSingleNaluPayload() const {
   rtc::CopyOnWriteBuffer res;
   auto& h264_header = H264Header();
   res.AppendData(&h264_header.nalus[0].type, 1);
-  res.AppendData(nalu_payloads_[0].data(), nalu_payloads_[0].size());
+  res.AppendData(nalu_payloads_[0]);
   return res;
 }
 
@@ -224,7 +223,7 @@
     res.AppendData(length_as_array);
 
     res.AppendData(&h264_header.nalus[i].type, 1);
-    res.AppendData(nalu_payloads_[i].data(), nalu_payloads_[i].size());
+    res.AppendData(nalu_payloads_[i]);
   }
   return res;
 }