From ecb6bc735c15a6ea7be0fb2681b18397c9e3f0b2 Mon Sep 17 00:00:00 2001 From: Abe Jellinek Date: Wed, 2 Jul 2025 15:36:09 -0400 Subject: [PATCH] Update feedReader.js hiddenDOMWindow comment --- chrome/content/zotero/xpcom/feedReader.js | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/chrome/content/zotero/xpcom/feedReader.js b/chrome/content/zotero/xpcom/feedReader.js index ecc39a6fda..5f484d2f67 100644 --- a/chrome/content/zotero/xpcom/feedReader.js +++ b/chrome/content/zotero/xpcom/feedReader.js @@ -125,11 +125,8 @@ Zotero.FeedReader = function (url) { }.bind(this)); // The feed processor and related modules assume a content window environment, so we'll simulate - // one via a sandbox in a parent window. You might think we could jump straight to - // `hiddenDOMWindow` as a parent window, since it does indeed exist on all platforms... - // However, when loading scripts into the `hiddenDOMWindow` on Windows and Linux, they get - // stuck in some lazily parsed state which bizarrely drops function prototypes. To avoid this, - // we prefer other parent windows first, which work fine on all platforms. + // one via a sandbox in a parent window. hiddenDOMWindow doesn't exist outside macOS, so we + // prefer other parent windows first, which work fine on all platforms. let parentWindow = Services.wm.getMostRecentWindow("navigator:browser"); if (!parentWindow) { parentWindow = Services.ww.activeWindow;