Ehmmm... non sono sicuro che la sintassi sia corretta, ma io gestirei il problema in un modo molto più semplice:
codice:
function flash() {
  document.getElementById("testo").style.visibility = ~ document.getElementById("testo").style.visibility;
}

setTimeout("flash()",200);
In questo modo dovresti evitare di gestire lo statement "if...else" in quanto ogni 200 millisecondi la visibilità dovrebbe cambiare indipendentemente dal sua valore (grazie all'operatore NOT).

Tutto molto bello ma... non sono sicuro che funzioni ;)

Secan