Deprecate RtcEventLogFactory constructor taking unused parameter Bug: webrtc:15656 Change-Id: I22ed4cca4c0ce7ebf9c533ed7434617bf0a0f4a3 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/330120 Reviewed-by: Harald Alvestrand <hta@webrtc.org> Commit-Queue: Danil Chapovalov <danilchap@webrtc.org> Cr-Commit-Position: refs/heads/main@{#41338}
diff --git a/api/create_peerconnection_factory.cc b/api/create_peerconnection_factory.cc index 5d3aace..bd77f74 100644 --- a/api/create_peerconnection_factory.cc +++ b/api/create_peerconnection_factory.cc
@@ -48,8 +48,7 @@ dependencies.signaling_thread = signaling_thread; dependencies.task_queue_factory = CreateDefaultTaskQueueFactory(field_trials.get()); - dependencies.event_log_factory = std::make_unique<RtcEventLogFactory>( - dependencies.task_queue_factory.get()); + dependencies.event_log_factory = std::make_unique<RtcEventLogFactory>(); dependencies.trials = std::move(field_trials); if (network_thread) {
diff --git a/api/rtc_event_log/rtc_event_log_factory.h b/api/rtc_event_log/rtc_event_log_factory.h index 21a670e..1deb061 100644 --- a/api/rtc_event_log/rtc_event_log_factory.h +++ b/api/rtc_event_log/rtc_event_log_factory.h
@@ -26,10 +26,9 @@ public: RtcEventLogFactory() = default; - // TODO(bugs.webrtc.org/15656): deprecate and delete constructor taking - // task queue factory in favor of using task queue factory provided through - // the Environment parameter in Create function. + [[deprecated("Use default constructor")]] // explicit RtcEventLogFactory(TaskQueueFactory* task_queue_factory) {} + ~RtcEventLogFactory() override = default; absl::Nonnull<std::unique_ptr<RtcEventLog>> Create(