fx-compat: Fix basicViewer for http(s):// URLs

maychangeremoteness="true" is required to successfully load pages from the web even though we never actually change remoteness. https://searchfox.org/comm-central/rev/2cf3a09a0127fe73e932c5ee793986f3bed2ccd2/mail/base/content/aboutMessage.xhtml#118
zotero · May 26, 2022 · f05d6fe · f05d6fe
1 parent e26fd18
commit f05d6fe
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/chrome/content/zotero/standalone/basicViewer.js b/chrome/content/zotero/standalone/basicViewer.js
@@ -48,10 +48,10 @@ window.addEventListener("load", /*async */function() {
 	//browser.docShellIsActive = false;
 
 	// align page title with title of shown document
-	browser.addEventListener("pageshow", function() {
-		document.title = browser.contentDocument.title || browser.contentDocument.location.href;
-	}, false);
-	
+	browser.addEventListener('pagetitlechanged', () => {
+		document.title = browser.contentTitle || browser.currentURI.spec;
+	});
+
 	// Load URI passed in as nsISupports .data via openWindow()
 	browser.loadURI(
 		window.arguments[0],

diff --git a/chrome/content/zotero/standalone/basicViewer.xhtml b/chrome/content/zotero/standalone/basicViewer.xhtml
@@ -170,7 +170,8 @@
 					type="content"
 					flex="1"
 					remote="false"
-					disableglobalhistory="true"/>
+					disableglobalhistory="true"
+					maychangeremoteness="true"/>
 		</vbox>
 	</hbox>
 </window>