C'e` un errore:
anziche`
if(popup)

devi scrivere:
if(!popup.closed)

oppure (solo in questo caso, dato che la variabile viene settata a null):
if(popup==null)

ma forse la cosa migliore e`:
if(popup && !popup.closed)

Ciao
Michele