From bb0d7ab95140839c20ffe2e5e1d3fa7120120934 Mon Sep 17 00:00:00 2001 From: Abe Jellinek Date: Tue, 26 Nov 2024 10:16:49 -0500 Subject: [PATCH] PageData actor: Settle for readyState "interactive" Some pages seemingly never become "complete". "interactive" (which we already use in TranslationChild) is equivalent to DOMContentLoaded, and that's enough for everything PageDataChild needs to do right now. This should make Scaffold test runs more reliable. TranslatorTester uses { requireSuccessfulStatus: true }, which calls the PageData actor, and that was occasionally flaking and causing the test to time out. Fixes #4869 --- chrome/content/zotero/actors/PageDataChild.jsm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/chrome/content/zotero/actors/PageDataChild.jsm b/chrome/content/zotero/actors/PageDataChild.jsm index c69de7982b..228003d509 100644 --- a/chrome/content/zotero/actors/PageDataChild.jsm +++ b/chrome/content/zotero/actors/PageDataChild.jsm @@ -81,7 +81,7 @@ class PageDataChild extends JSWindowActorChild { const document = this.document; function readyEnough() { - return document.readyState === "complete"; + return document.readyState === "complete" || document.readyState === "interactive"; } if (readyEnough()) {