From 8ed8a97ae9c4a43f0dd700e279e4bba7966037b2 Mon Sep 17 00:00:00 2001 From: daijro Date: Sun, 3 Nov 2024 01:38:44 -0500 Subject: [PATCH] Don't block setViewport When window.innerWidth or window.innerHeight is not provided, allow Juggler's setViewport to still work. --- additions/juggler/TargetRegistry.js | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/additions/juggler/TargetRegistry.js b/additions/juggler/TargetRegistry.js index 99665a1..32eaf45 100644 --- a/additions/juggler/TargetRegistry.js +++ b/additions/juggler/TargetRegistry.js @@ -383,11 +383,17 @@ class PageTarget { this._tab = tab; this._linkedBrowser = tab.linkedBrowser; this._browserContext = browserContext; + this._viewportSize = undefined; // Set the viewport size to Camoufox's default value. - this._viewportSize = { - width: ChromeUtils.camouGetInt("window.innerWidth") || 1280, - height: ChromeUtils.camouGetInt("window.innerHeight") || 720, - };; + if ( + ChromeUtils.camouGetInt("window.innerWidth") + || ChromeUtils.camouGetInt("window.innerHeight") + ) { + this._viewportSize = { + width: ChromeUtils.camouGetInt("window.innerWidth") || 1280, + height: ChromeUtils.camouGetInt("window.innerHeight") || 720, + }; + } this._initialDPPX = this._linkedBrowser.browsingContext.overrideDPPX; this._url = 'about:blank'; this._openerId = opener ? opener.id() : undefined;