improve and test the dafult policy dialog

This commit also fixes some bugs that manifested themselves spuriously.

2 files changed, 3 insertions, 2 deletions

diff --git a/background/broadcast_broker.js b/background/broadcast_broker.js
index 5af3a7b..458fb7f 100644
--- a/background/broadcast_broker.js
+++ b/background/broadcast_broker.js
@@ -54,7 +54,8 @@ function new_broadcast_listener(port)
 {
     const listener_ctx = {port, id: ++next_id, channels: new Set()};
     port.onMessage.addListener(msg => listener_command(msg, listener_ctx));
-    port.onDisconnect.addListener(msg => listener_remove(msg, listener_ctx));
+    const on_disconnect = () => remove_broadcast_listener(listener_ctx);
+    port.onDisconnect.addListener(on_disconnect);
 }
 
 function listener_command(msg, listener_ctx)
diff --git a/background/webrequest.js b/background/webrequest.js
index e32947a..9787318 100644
--- a/background/webrequest.js
+++ b/background/webrequest.js
@@ -74,7 +74,7 @@ function on_headers_received(details)
 
     let headers = details.responseHeaders;
 
-    const policy = decide_policy(tree, details.url, default_allow, secret);
+    const policy = decide_policy(tree, details.url, !!default_allow, secret);
     if (policy.allow)
 	return;