summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjanekptacijarabaci <janekptacijarabaci@seznam.cz>2018-03-01 13:50:06 +0100
committerjanekptacijarabaci <janekptacijarabaci@seznam.cz>2018-03-01 13:50:06 +0100
commitcaebf13372415e59bd36c49fbee934e25ec46a32 (patch)
tree62117dd9cb7e0b119f8b54910aafa9c1023f824b
parent66e6aac50638d1f706f0ddaee1529a7b3cb47075 (diff)
downloadUXP-caebf13372415e59bd36c49fbee934e25ec46a32.tar
UXP-caebf13372415e59bd36c49fbee934e25ec46a32.tar.gz
UXP-caebf13372415e59bd36c49fbee934e25ec46a32.tar.lz
UXP-caebf13372415e59bd36c49fbee934e25ec46a32.tar.xz
UXP-caebf13372415e59bd36c49fbee934e25ec46a32.zip
Checking NS_BINDING_ABORTED to avoid dispatching navigate event
https://github.com/MoonchildProductions/moebius/pull/302
-rw-r--r--devtools/server/actors/webbrowser.js6
1 files changed, 5 insertions, 1 deletions
diff --git a/devtools/server/actors/webbrowser.js b/devtools/server/actors/webbrowser.js
index 1808895b1..dffe49b91 100644
--- a/devtools/server/actors/webbrowser.js
+++ b/devtools/server/actors/webbrowser.js
@@ -2501,7 +2501,11 @@ DebuggerProgressListener.prototype = {
if (isWindow && isStop) {
// Don't dispatch "navigate" event just yet when there is a redirect to
// about:neterror page.
- if (request.status != Cr.NS_OK) {
+ // Navigating to about:neterror will make `status` be something else than NS_OK.
+ // But for some error like NS_BINDING_ABORTED we don't want to emit any `navigate`
+ // event as the page load has been cancelled and the related page document is going
+ // to be a dead wrapper.
+ if (request.status != Cr.NS_OK && request.status != Cr.NS_BINDING_ABORTED) {
// Instead, listen for DOMContentLoaded as about:neterror is loaded
// with LOAD_BACKGROUND flags and never dispatches load event.
// That may be the same reason why there is no onStateChange event