From 2f621ba95a51f30dace4b270d75eb543ae03ebbf Mon Sep 17 00:00:00 2001 From: "D. Seifert" Date: Fri, 5 Jul 2024 03:52:58 +0800 Subject: [PATCH] Assert SequenceManagerImpl::AdjustWakeUp --- .../sequence_manager/sequence_manager_impl.cc | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/base/task/sequence_manager/sequence_manager_impl.cc b/base/task/sequence_manager/sequence_manager_impl.cc index f1aa9deb51ec0e..f6f0422483076c 100644 --- a/base/task/sequence_manager/sequence_manager_impl.cc +++ b/base/task/sequence_manager/sequence_manager_impl.cc @@ -563,10 +563,6 @@ void SequenceManagerImpl::SetNextWakeUp(LazyNow* lazy_now, absl::optional wake_up) { auto next_wake_up = AdjustWakeUp(wake_up, lazy_now); - recordreplay::Assert( - "[RUN-2801-2978] SequenceManagerImpl::SetNextWakeUp %d", - next_wake_up && next_wake_up->is_immediate()); - if (next_wake_up && next_wake_up->is_immediate()) { ScheduleWork(); } else { @@ -828,6 +824,13 @@ absl::optional SequenceManagerImpl::AdjustWakeUp( absl::optional wake_up, LazyNow* lazy_now) const { DCHECK_CALLED_ON_VALID_THREAD(associated_thread_->thread_checker); + + REPLAY_ASSERT_MAYBE_EVENTS_DISALLOWED( + "[TT-1504-1505] SequenceManagerImpl::AdjustWakeUp A %d %d %llu", + !!wake_up, + wake_up && wake_up->is_immediate(), + wake_up->earliest_time().ToInternalValue()); + if (!wake_up) return absl::nullopt; @@ -838,6 +841,10 @@ absl::optional SequenceManagerImpl::AdjustWakeUp( // up scheduled, so pretend we have no more work. This will result in // appearing idle and |time_domain| will decide what to do in // MaybeFastForwardToWakeUp(). + REPLAY_ASSERT_MAYBE_EVENTS_DISALLOWED( + "[TT-1504-1505] SequenceManagerImpl::AdjustWakeUp B %d", + !!main_thread_only().time_domain); + if (main_thread_only().time_domain) return absl::nullopt; return *wake_up;