diff --git a/platform/firefox/frameModule.js b/platform/firefox/frameModule.js index cee9af609..f258a2d8d 100644 --- a/platform/firefox/frameModule.js +++ b/platform/firefox/frameModule.js @@ -294,10 +294,12 @@ const contentObserver = { let doc = e.target; doc.removeEventListener(e.type, docReady, true); + // It is possible, in some cases (#1140) for document-element-inserted to occur *before* nsIWebProgressListener.onLocationChange, so ensure that the URL is correct before continuing let messageManager = doc.docShell.getInterface(Ci.nsIContentFrameMessageManager); messageManager.sendSyncMessage(locationChangedMessageName, { url: loc.href, + noRefresh: true, // If the URL is the same, then don't refresh it so that if this occurs after onLocationChange, no the block count isn't reset }); lss(this.contentBaseURI + 'contentscript-end.js', sandbox); diff --git a/platform/firefox/vapi-background.js b/platform/firefox/vapi-background.js index c5c457d7c..982a0c54f 100644 --- a/platform/firefox/vapi-background.js +++ b/platform/firefox/vapi-background.js @@ -1325,6 +1325,12 @@ vAPI.net.registerListeners = function() { var locationChangedListener = function(e) { var details = e.data; var browser = e.target; + + if (details.noRefresh && details.url === browser.currentURI.asciiSpec) { // If the location changed message specified not to refresh, and the URL is the same, no need to do anything + //console.debug("nsIWebProgressListener: ignoring onLocationChange: " + details.url); + return; + } + var tabId = vAPI.tabs.getTabId(browser); if (tabId === vAPI.noTabId) { return; // Do not navigate for behind the scenes