Bug 1785549, ensure requestIdleCallback's callback doesn't get deadli…

…nes > 50ms, r=mccr8 Differential Revision: https://phabricator.services.mozilla.com/D155867
dothq · Aug 29, 2022 · fa09370 · fa09370
1 parent 00b74d2
commit fa09370
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/dom/base/IdleDeadline.cpp b/dom/base/IdleDeadline.cpp
@@ -60,7 +60,9 @@ DOMHighResTimeStamp IdleDeadline::TimeRemaining() {
       return 0.0;
     }
 
-    return std::max(mDeadline - performance->Now(), 0.0);
+    // The web API doesn't expect deadlines > 50ms, but conversion from the
+    // internal API may lead to some rounding errors.
+    return std::min(std::max(mDeadline - performance->Now(), 0.0), 50.0);
   }
 
   // If there's no window, we're in a system scope, and can just use

diff --git a/testing/web-platform/meta/requestidlecallback/deadline-max.html.ini b/testing/web-platform/meta/requestidlecallback/deadline-max.html.ini