Нужна помощь по js + html

Скажу сразу js не знаю вовсе пытаюсь разобраться на практике сочиняя расширения для Хрома

Где то украл где то подсказали, итог вот этот код:

HTML popup

Цитата:

<script type="text/javascript" src="js/info.js"></script>

<form id="topform">
<input type="checkbox" name="f1" onchange="noindex()"><label>Показать NoIndex</label>
</form>

скрипт чужой и по идее рабочий

Цитата:

(function noindex(){

var indicator = 1; // чем помечать внешние ссылки, 0 - рамка (border), 1 - цвет фона (background)
var color_ind = "#00ff18"; // цвет метки внешних ссылок

// отступы необходимо установить, если выбран индикатор - цвет фона, иначе не будет видно внешнюю ссылку с анкором - картинкой
var padding_left_ind = "10px";
var padding_right_ind = "10px";

var border_width = "2px"; // толщина рамки

var noindex = ""; // как помечать noindex ссылки
var color_ni = "#f00"; // цвет метки noindex ссылок
var nofollow = ""; // как помечать nofollow ссылки
var color_nf = "#00f"; // цвет метки nofollow ссылок

var x = document.getElementsByTagName("a" ) ;
for(var a in x){
if(x.href&&(!x.href.match("http:\/\/(www.)?"+ document.domain.replace("www.", "" ) +"|@|javascript:|about:|opera:|widget:|resource:|mailto:|file:" ))){
if(indicator==0) x.innerHTML = "<span style=\"border:" + color_ind + " " + border_width + " solid;\">" + x.innerHTML + "</span>";
if(indicator==1) x.innerHTML = "<span style=\"padding-left:" + padding_left_ind + ";padding-right:" + padding_right_ind + ";background-color:" + color_ind + ";\">" + x.innerHTML + "</span>";
}
}

var y = document.getElementsByTagName("noindex" ) ;
for(var b in y){
var i = y.getElementsByTagName("a" ) ;
for(var j in i){
i.innerHTML += "<span style=\"background-color:#fff;font:Arial;font-size:10px;font-weight:bold; color:" + color_ni + ";text-decoration:none;\"> " + noindex + "</span>";
}
}

var z = document.getElementsByTagName("a" ) ;
for(var c in z){
if(z.rel.match("nofollow" ) ){
z.innerHTML += "<span style=\"background-color:#fff;font:Arial;font-size:10px;font-weight:bold; color:" + color_nf + ";text-decoration:none;\"> " + nofollow + "</span>";
}
}

})();

Итог в том, что ничего не работает )))) кто подскажет в чем моя ошибка, я уверен что просто где то ошибся в синтаксисе.

Спасибо всем кто хотя бы осилил этот несвязный пост)

shatamba, чем ковырять такой код - проще написать свой. Чесслово.

shatamba, учите лучше Java Script.

Я понимаю что лучше учить Java Script этим и занимаюсь на интересном мне примере, но хотелось бы получить что то более по делу, правильный ли синтаксис? Насколько верна логика подхода? В какую сторону копать?

Цитата ("shatamba"):

что то более по делу

Можно начать с этого: посилання
Цитата ("shatamba"):

правильный ли синтаксис?

Ссылка что выше тоже поможет.
Цитата ("shatamba"):

насколько верна логика подхода

А вот тут уже надо самому установить и подключить head.dll
Цитата ("shatamba"):

в какую сторону копать?

Точно могу сказать что начали копать не в ту сторону.

Уберите в начале скобку, и в конце скобку, () и ;

Попробуйте еще там где функция объявляется заменить на onchange="javascript: noindex();"