Mark the capture time identifier fields deprecated
These would be removed in favour of presentation timestamp field.
Bug: webrtc:373365537
Change-Id: I276f8cfe66373fa9332a2722f28832fc5cdb935f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/380702
Reviewed-by: Guido Urdaneta <guidou@webrtc.org>
Commit-Queue: Palak Agarwal <agpalak@google.com>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#44232}
diff --git a/api/frame_transformer_interface.h b/api/frame_transformer_interface.h
index 2bcc32d..fd53f30 100644
--- a/api/frame_transformer_interface.h
+++ b/api/frame_transformer_interface.h
@@ -57,7 +57,9 @@
// TODO(https://bugs.webrtc.org/373365537): Remove this once its usage is
// removed from blink.
- virtual std::optional<Timestamp> GetCaptureTimeIdentifier() const {
+ [[deprecated(
+ "Use GetPresentationTimestamp instead")]] virtual std::optional<Timestamp>
+ GetCaptureTimeIdentifier() const {
return std::nullopt;
}
diff --git a/api/video/video_frame.h b/api/video/video_frame.h
index 5ddee8b..d4a9b1c 100644
--- a/api/video/video_frame.h
+++ b/api/video/video_frame.h
@@ -108,7 +108,8 @@
const rtc::scoped_refptr<VideoFrameBuffer>& buffer);
Builder& set_timestamp_ms(int64_t timestamp_ms);
Builder& set_timestamp_us(int64_t timestamp_us);
- Builder& set_capture_time_identifier(
+ [[deprecated("Use set_presentation_timestamp instead")]] Builder&
+ set_capture_time_identifier(
const std::optional<Timestamp>& presentation_timestamp);
Builder& set_presentation_timestamp(
const std::optional<Timestamp>& presentation_timestamp);