Michael Froman
20fab29478
Upstream commit: https://webrtc.googlesource.com/src/+/26040c0284e01b031be86a7b792bb523f07bd00a Move video timing worker objects to `//video/timing`. Tested: Verified that the moved unit tests showed up in https://luci-milo.appspot.com/ui/inv/task-chromium-swarm.appspot.com-79314841a24dad11/test-results Bug: b/493549134 Change-Id: I3d8a4ba98498539c893aed3ac5cadf8058ec3b23 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/485840 Reviewed-by: Åsa Persson <asapersson@webrtc.org> Commit-Queue: Rasmus Brandt <brandtr@webrtc.org> Cr-Commit-Position: refs/heads/main@{#48135}
24 lines
1.0 KiB
Diff
24 lines
1.0 KiB
Diff
From: Andreas Pehrson <apehrson@mozilla.com>
|
|
Date: Wed, 11 Jan 2023 22:42:00 +0000
|
|
Subject: Bug 1800942 - Add DCHECKs to
|
|
TimestampExtrapolator::ExtrapolateLocalTime. r=mjf
|
|
|
|
Differential Revision: https://phabricator.services.mozilla.com/D166536
|
|
Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/c5df7f40392464ffc63f44a53ddcaab2091741e0
|
|
---
|
|
video/timing/timestamp_extrapolator.cc | 1 +
|
|
1 file changed, 1 insertion(+)
|
|
|
|
diff --git a/video/timing/timestamp_extrapolator.cc b/video/timing/timestamp_extrapolator.cc
|
|
index 77eb9d6e13..e6e8dd3d8d 100644
|
|
--- a/video/timing/timestamp_extrapolator.cc
|
|
+++ b/video/timing/timestamp_extrapolator.cc
|
|
@@ -265,6 +265,7 @@ void TimestampExtrapolator::Update(Timestamp now, uint32_t ts90khz) {
|
|
std::optional<Timestamp> TimestampExtrapolator::ExtrapolateLocalTime(
|
|
uint32_t timestamp90khz) const {
|
|
int64_t unwrapped_ts90khz = unwrapper_.PeekUnwrap(timestamp90khz);
|
|
+ RTC_DCHECK_GE(unwrapped_ts90khz, 0);
|
|
|
|
if (!first_unwrapped_timestamp_) {
|
|
return std::nullopt;
|