Hola otra vez:
Donde pone:
popup = window.open(...);
Hay que cambiar los puntos suspensivos por lo que se debe poner dentro del open...
Hay información muy detallada de los parámetros de los popus en la FAQ 100 de javascript.
El primer parámetro es la página destino, el segundo el nombre de la ventana y el tercero la lista de opciones de la ventana.
Saludos