diff --git a/platform/firefox/frameModule.js b/platform/firefox/frameModule.js index aab44730b..f258a2d8d 100644 --- a/platform/firefox/frameModule.js +++ b/platform/firefox/frameModule.js @@ -293,6 +293,15 @@ const contentObserver = { let docReady = (e) => { 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); if ( doc.querySelector('a[href^="abp:"]') ) { 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