if (objSel.options[i].text == cityname){
objSel.selectedIndex = i;
selectcity = objSel.options[i].text;//выставляем город в селекторе
break}
просто в вас получается бесконечный while т.к. вы его не прерываете
на отрицание поставьте objSel.options[i]==0 или break тоесть ваша задача забить значением это переменную дать ей значение/длинну что бы objSel.options[i] - существовала
else {ТУТ кОД ПРЕРЫВАНИЯ или ЗАБИВАЕМ ЗНАЧЕНИЕ\ДЛИННУ}