From 58ee9dff08dfd2534be54f058ca3f726668b83d4 Mon Sep 17 00:00:00 2001 From: Tony Herre Date: Mon, 26 Jun 2023 15:30:06 +0000 Subject: [PATCH] Deprecate encoded audio frame GetHeader Bug: chromium:1456628 Change-Id: Ifc7d1aa1153c0593c673381f153e5793b94c98c3 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/310420 Commit-Queue: Harald Alvestrand Auto-Submit: Tony Herre Reviewed-by: Harald Alvestrand Cr-Commit-Position: refs/heads/main@{#40359} --- api/frame_transformer_interface.h | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/api/frame_transformer_interface.h b/api/frame_transformer_interface.h index 47058d515f..73d27027cc 100644 --- a/api/frame_transformer_interface.h +++ b/api/frame_transformer_interface.h @@ -69,12 +69,10 @@ class TransformableAudioFrameInterface : public TransformableFrameInterface { virtual ~TransformableAudioFrameInterface() = default; virtual void SetRTPTimestamp(uint32_t timestamp) = 0; - // Exposes the frame header, enabling the interface clients to use the - // information in the header as needed, for example to compile the list of - // csrcs. - // TODO(crbug.com/1453226): Deprecate and remove once callers have migrated to - // the getters for specific fields. - virtual const RTPHeader& GetHeader() const = 0; + + // TODO(crbug.com/1453226): Remove after a few weeks. + [[deprecated("Use specific getters instead.")]] virtual const RTPHeader& + GetHeader() const = 0; virtual rtc::ArrayView GetContributingSources() const = 0;