Bug 320275 !Node "compat" hack is very broken

r=myk a=justdave

1 files changed, 10 insertions, 8 deletions

diff --git a/template/en/default/bug/dependency-tree.html.tmpl b/template/en/default/bug/dependency-tree.html.tmpl
index b8e0c79e1..8b5ecd59c 100644
--- a/template/en/default/bug/dependency-tree.html.tmpl
+++ b/template/en/default/bug/dependency-tree.html.tmpl
@@ -109,9 +109,11 @@
 [% END %]
 
 <script type="text/javascript">
-if (!Node) {
-    /* MSIE doesn't define Node, so provide a compatibility array */
-    var Node = { TEXT_NODE: 3 };
+if (typeof Node == 'undefined') {
+    /* MSIE doesn't define Node, so provide a compatibility object */
+    window.Node = {
+        TEXT_NODE: 3
+    };
 }
 
 function toggleDisplay(node)
@@ -119,7 +121,7 @@ function toggleDisplay(node)
     var display = node.style.display;
     if (display == "none") {
         node.style.display =
-           ("oldDisplay" in node) ? node.oldDisplay : "block";
+            ("oldDisplay" in node) ? node.oldDisplay : "block";
         return true;
     }
 
@@ -132,18 +134,18 @@ function listToggle(event)
 {
     var node = event.target;
     if (!node)
-        node = event.srcElement;
+       node = event.srcElement;
     if (node.nodeType == Node.TEXT_NODE)
         node = node.parentNode;
     var toggle = node.nextSibling;
     while (toggle && toggle.tagName != "UL")
-      toggle = toggle.nextSibling;
+        toggle = toggle.nextSibling;
     if (toggle) {
-      node.firstChild.data = toggleDisplay(toggle) ? "[-]" : "[+]";
+        node.firstChild.data = toggleDisplay(toggle) ? "[-]" : "[+]";
     }
 }
 
- </script>
+</script>
 
 [% PROCESS global/footer.html.tmpl %]