Empty host (e.g. file:/// URIs) should also mean that the menu item i…

…sn't shown.
robsonsobral · May 12, 2010 · 7a07985 · 7a07985
1 parent 19d7bbb
commit 7a07985
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/modules/prototypes.js b/modules/prototypes.js
@@ -203,8 +203,11 @@ BarTabHandler.prototype = {
             host = tab.linkedBrowser.currentURI.host;
         } catch (ex) {
             // Most likely uri.host doesn't exist which probably means
-            // whitelisting doesn't make sense on this tab.  Don't
-            // show the menu item
+            // whitelisting doesn't make sense on this tab.  Set empty
+            // host so we don't show the menu item
+            host = '';
+        }
+        if (!host) {
             neverunload.setAttribute("hidden", "true");
             unloadtab.removeAttribute("disabled");
             return;