commit 71ba9cbb80b39769524798cbd6eff5f1c062969c
author Emil Vardar <vardar@google.com> Fri Nov 08 15:27:34 2024
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Nov 11 10:03:37 2024
tree f8b8fdeb685163b536fe4b08081e3baeb00b0f64
parent 849549d40304890255f6033d13c2646e3bfb6ebb

Add corruption detection header default as stopped.

Bug: webrtc:358039777
Change-Id: I957638c4a84f26391b09af677cc7aaf2bf2024ee
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/368000
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Emil Vardar (xWF) <vardar@google.com>
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#43385}

media/engine/webrtc_video_engine.cc

diff --git a/media/engine/webrtc_video_engine.cc b/media/engine/webrtc_video_engine.cc
index 996f0d7..781e997 100644
--- a/media/engine/webrtc_video_engine.cc
+++ b/media/engine/webrtc_video_engine.cc
@@ -846,6 +846,9 @@
         webrtc::RtpExtension::kRidUri, webrtc::RtpExtension::kRepairedRidUri}) {
     result.emplace_back(uri, id++, webrtc::RtpTransceiverDirection::kSendRecv);
   }
+  result.emplace_back(webrtc::RtpExtension::kCorruptionDetectionUri, id++,
+                      /*preferred_encrypt=*/true,
+                      webrtc::RtpTransceiverDirection::kStopped);
   for (const auto& uri : {webrtc::RtpExtension::kAbsoluteCaptureTimeUri}) {
     result.emplace_back(uri, id, webrtc::RtpTransceiverDirection::kStopped);
   }

media/engine/webrtc_video_engine_unittest.cc

diff --git a/media/engine/webrtc_video_engine_unittest.cc b/media/engine/webrtc_video_engine_unittest.cc
index 913560a..3aff1a6 100644
--- a/media/engine/webrtc_video_engine_unittest.cc
+++ b/media/engine/webrtc_video_engine_unittest.cc
@@ -507,6 +507,10 @@
   ExpectRtpCapabilitySupport(RtpExtension::kGenericFrameDescriptorUri00, false);
 }
 
+TEST_F(WebRtcVideoEngineTest, SupportCorruptionDetectionHeaderExtension) {
+  ExpectRtpCapabilitySupport(RtpExtension::kCorruptionDetectionUri, false);
+}
+
 class WebRtcVideoEngineTestWithGenericDescriptor
     : public WebRtcVideoEngineTest {
  public:
@@ -3176,6 +3180,9 @@
 TEST_F(WebRtcVideoChannelTest, SendCorruptionDetectionHeaderExtensions) {
   TestSetSendRtpHeaderExtensions(RtpExtension::kCorruptionDetectionUri);
 }
+TEST_F(WebRtcVideoChannelTest, RecvCorruptionDetectionHeaderExtensions) {
+  TestSetRecvRtpHeaderExtensions(RtpExtension::kCorruptionDetectionUri);
+}
 
 TEST_F(WebRtcVideoChannelTest, DisableFrameInstrumentationByDefault) {
   EXPECT_TRUE(send_channel_->SetSenderParameters(send_parameters_));