expand menu problem

Jeg har lavet følgende script:

var ns, ns6, ie;
ns  = (document.layers) ? true : false;
ns6 = (document.getElementById) ? true : false;
ie  = (document.all) ? true : false;

function ExpandLayer(layer) {
    if(ns) {
        if (document.layers[layer].visibility == "hidden") {
            document.layers[layer].visibility = "show";
        } else {
            document.layers[layer].visibility = "hidden";
        }
    } else if(ie) {
        if (document.all[layer].style.visibility == "hidden") {
            document.all[layer].style.visibility = "visible";
        } else {
            document.all[layer].style.visibility = "hidden";
        }
    } else if(ns6) {
        if (document.getElementById(layer).style.visibility == "hidden") {
            document.getElementById(layer).style.visibility = "visible";
        } else {
            document.getElementById(layer).style.visibility = "hidden";
        }
    }
}

<tr onMouseOver="this.className='over'" onMouseOut="this.className='norm'" onClick="java script:ExpandLayer('mailbox');">
  <td class="MenuItem"><img src="images/pil.gif"> Mailbox</td>
</tr>
<div style="visibility:hidden" id="mailbox">
  <tr onMouseOver="this.className='over'" onMouseOut="this.className='norm'" onClick="location.href='mailbox.asp'">
    <td>&nbsp;&nbsp;<img src="images/pil.gif"> Indbakke</td>
  </tr>
</div>

html'en har jeg bare lige taget fra mit eget script, så jeg ved godt at der mangler en start og en slut table.
Mit problem er at div'en er synlig hele tiden, det er jo meningen at når man trykker på td'en så skal div'en blive synlig.
Der sker heller ikke noget når jeg trykker på td'en, det er som om at der mangler noget i javascriptet, men jeg kan ikke se hvad der skulle mangle. jeg synes det ser rigtigt nok ud, men jeg arbejder heller ikke så tit med javascript, så det kan jo godt være jeg har glemt noget.

På forhånd tak!