evita usar las referencias directas, usa en cambio las colecciones, es decir esto:
document.form1.clave.value == ''
por esto:
document.
forms['form1'].
elements['clave'].value == ''
por otro lado avalúas al principio
document.form1.value == '" y hasta donde sé el
form no tiene value
