Use 'load' instead of 'pageshow' in Zotero.HTTP.loadDocuments()

This seems to fix snapshots in Twitter, which need to wait for a
JS-based self-redirect [1].

[1] https://blog.twitter.com/engineering/en_us/topics/insights/2018/twitter_silhouette.html
This commit is contained in:
Dan Stillman 2018-09-21 19:48:54 -04:00
parent 2957db2aa1
commit c6a2af7ab5

View file

@ -914,7 +914,7 @@ Zotero.HTTP = new function() {
}
Zotero.debug("Zotero.HTTP.loadDocuments: " + url + " loaded");
hiddenBrowser.removeEventListener("pageshow", onLoad, true);
hiddenBrowser.removeEventListener("load", onLoad, true);
hiddenBrowser.zotero_loaded = true;
var maybePromise;
@ -962,7 +962,7 @@ Zotero.HTTP = new function() {
for(var i=0; i<urls.length; i++) {
var hiddenBrowser = Zotero.Browser.createHiddenBrowser();
if(cookieSandbox) cookieSandbox.attachToBrowser(hiddenBrowser);
hiddenBrowser.addEventListener("pageshow", onLoad, true);
hiddenBrowser.addEventListener("load", onLoad, true);
hiddenBrowsers[i] = hiddenBrowser;
}