shikuwa, вам в списке с
id=frame нужно указать № сооветствующих фреймов, а не подставлять неизвестные значения...
parent["fr"+fr].document.getElementById("img").src = img;
<select id="frame">
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
</select>
и имена сделайте у всех фреймов
frX
один закрывающий тег
</frameset> лишний