if (this.src="1.jpg")
должно быть
if (this.src=="1.jpg")
хотя конечно лучше, как написал Phoenix, только и там можно схитрить:
function chg(id){
var el = document.getElementById(id);
if (el.src.indexOf("1.jpg")>0){
el.src="3.jpg"
}else{
el="1.jpg"
}
}
Зачем лишние два раза дом дергать
