Use Timestamp and TimeDelta in VCMTiming
* Switches TimestampExtrapolator to use Timestamp as well.
Bug: webrtc:13589
Change-Id: I042be5d693068553d2e8eb92fa532092d77bd7ef
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/249993
Reviewed-by: Tomas Gunnarsson <tommi@webrtc.org>
Commit-Queue: Evan Shrubsole <eshr@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#36112}
diff --git a/video/frame_decode_timing_unittest.cc b/video/frame_decode_timing_unittest.cc
index ac5bf9e..0d99847 100644
--- a/video/frame_decode_timing_unittest.cc
+++ b/video/frame_decode_timing_unittest.cc
@@ -32,20 +32,18 @@
public:
explicit FakeVCMTiming(Clock* clock) : webrtc::VCMTiming(clock) {}
- int64_t RenderTimeMs(uint32_t frame_timestamp,
- int64_t now_ms) const override {
+ Timestamp RenderTime(uint32_t frame_timestamp, Timestamp now) const override {
RTC_DCHECK(render_time_map_.contains(frame_timestamp));
auto it = render_time_map_.find(frame_timestamp);
- return it->second.ms();
+ return it->second;
}
- int64_t MaxWaitingTime(int64_t render_time_ms,
- int64_t now_ms,
- bool too_many_frames_queued) const override {
- auto render_time = Timestamp::Millis(render_time_ms);
+ TimeDelta MaxWaitingTime(Timestamp render_time,
+ Timestamp now,
+ bool too_many_frames_queued) const override {
RTC_DCHECK(wait_time_map_.contains(render_time));
auto it = wait_time_map_.find(render_time);
- return it->second.ms();
+ return it->second;
}
void SetTimes(uint32_t frame_timestamp,
