Skip to content

Commit

Permalink
Unreviewed, reverting 266621@main.
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=259876

Caused ~1% Speedometer 2.1 regression on macOS Sonoma

Reverted changeset:

"makeOpportunisticTaskDeferralScopeIfPossible has a wrong early exit logic"
https://bugs.webkit.org/show_bug.cgi?id=259857
https://commits.webkit.org/266621@main

Canonical link: https://commits.webkit.org/266627@main
  • Loading branch information
webkit-commit-queue authored and rniwa committed Aug 6, 2023
1 parent 33c467b commit b6f5d6e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Source/WebCore/page/DOMTimer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -427,7 +427,7 @@ std::optional<MonotonicTime> DOMTimer::alignedFireTime(MonotonicTime fireTime) c

void DOMTimer::makeOpportunisticTaskDeferralScopeIfPossible(ScriptExecutionContext& context)
{
if (!m_oneShot || m_currentTimerInterval > 1_ms) {
if (!m_oneShot || m_currentTimerInterval <= 1_ms) {
// FIXME: This should eventually account for repeating timers and one-shot timers that were
// previously scheduled, and are about to fire.
return;
Expand Down

0 comments on commit b6f5d6e

Please sign in to comment.