commit c6ce953664780563c873ac0dc8b306249740ac4b
author Harald Alvestrand <hta@webrtc.org> Fri Feb 24 10:57:54 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Feb 24 12:51:33 2023
tree 94682f266a5c4278106daf320066c4e22e86933a
parent 8566e779e3e6ddb58cd7a92504a17a2d96fb2c92

Add support for subclasses that override the old MediaChannel API.

Bug: webrtc:13931
Change-Id: I7d139c9914c5a4ac71b8d15fd4d73229d3331be2
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/294840
Reviewed-by: Florent Castelli <orphis@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#39387}

media/base/media_engine.h

diff --git a/media/base/media_engine.h b/media/base/media_engine.h
index 3891ddf..d3ef1f4 100644
--- a/media/base/media_engine.h
+++ b/media/base/media_engine.h
@@ -107,9 +107,17 @@
       const AudioOptions& options,
       const webrtc::CryptoOptions& crypto_options,
       webrtc::AudioCodecPairId codec_pair_id) {
+    // For the case where a subclass overrides the deprecated method
+    // but not the replacement method, call the deprecated method.
     // TODO(bugs.webrtc.org/13931): Remove default implementation
     // when downstream has migrated to new API.
-    RTC_CHECK_NOTREACHED();
+    RTC_CHECK(!recursion_guard_);
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+    RTC_LOG(LS_ERROR)
+        << "Override of deprecated declaration detected - please update!";
+    return CreateMediaChannel(call, config, options, crypto_options);
+#pragma clang diagnostic pop
   }
 
   // Backwards compatible version
@@ -118,9 +126,12 @@
                      const MediaConfig& config,
                      const AudioOptions& options,
                      const webrtc::CryptoOptions& crypto_options) {
-    return CreateMediaChannel(MediaChannel::Role::kBoth, call, config, options,
-                              crypto_options,
-                              webrtc::AudioCodecPairId::Create());
+    recursion_guard_ = true;
+    auto new_channel =
+        CreateMediaChannel(MediaChannel::Role::kBoth, call, config, options,
+                           crypto_options, webrtc::AudioCodecPairId::Create());
+    recursion_guard_ = false;
+    return new_channel;
   }
 
   virtual const std::vector<AudioCodec>& send_codecs() const = 0;
@@ -137,6 +148,11 @@
 
   virtual absl::optional<webrtc::AudioDeviceModule::Stats>
   GetAudioDeviceStats() = 0;
+
+ private:
+  // Workaround variable for avoiding recursion between old and new APIs.
+  // TODO(bugs.webrtc.org/13931): Remove when old interface is gone.
+  bool recursion_guard_ = false;
 };
 
 class VideoEngineInterface : public RtpHeaderExtensionQueryInterface {
@@ -156,10 +172,18 @@
       const VideoOptions& options,
       const webrtc::CryptoOptions& crypto_options,
       webrtc::VideoBitrateAllocatorFactory* video_bitrate_allocator_factory) {
+    // For the case where a subclass overrides the deprecated method
+    // but not the replacement method, call the deprecated method.
     // TODO(bugs.webrtc.org/13931): Remove default implementation
-    // when downstream has migrated.
-    RTC_CHECK_NOTREACHED();
-    return nullptr;
+    // when downstream has migrated to new API.
+    RTC_CHECK(!recursion_guard_);
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+    RTC_LOG(LS_ERROR)
+        << "Override of deprecated declaration detected - please update!";
+    return CreateMediaChannel(call, config, options, crypto_options,
+                              video_bitrate_allocator_factory);
+#pragma clang diagnostic pop
   }
 
   // Creates a video media channel.
@@ -172,8 +196,12 @@
       const VideoOptions& options,
       const webrtc::CryptoOptions& crypto_options,
       webrtc::VideoBitrateAllocatorFactory* video_bitrate_allocator_factory) {
-    return CreateMediaChannel(MediaChannel::Role::kBoth, call, config, options,
-                              crypto_options, video_bitrate_allocator_factory);
+    recursion_guard_ = true;
+    auto new_channel =
+        CreateMediaChannel(MediaChannel::Role::kBoth, call, config, options,
+                           crypto_options, video_bitrate_allocator_factory);
+    recursion_guard_ = false;
+    return new_channel;
   }
 
   // Retrieve list of supported codecs.
@@ -190,6 +218,11 @@
     RTC_DCHECK(include_rtx);
     return recv_codecs();
   }
+
+ private:
+  // Workaround variable for avoiding recursion between old and new APIs.
+  // TODO(bugs.webrtc.org/13931): Remove when old interface is gone.
+  bool recursion_guard_ = false;
 };
 
 // MediaEngineInterface is an abstraction of a media engine which can be