1. Вы получаете элемент из события различным способом для FF и IE, но забыли само получение объекта события распараллелить
e = window.event || e;
2. Так как событие вешается на каждый A и через on-атрибут, то вообще srcElement нет надобности получать - он будет this в функции.
3. А лучше не вешать обработчики на всё подряд, перебивая к тому же, возможно, уже существующие у каждого элемента, а повесить один:
(function() {
function handler(e)
{
e = e || window.event;
var target = e.target || e.srcElement;
if (target.tagName != "A") {
return true;
}
var href = target.getAttribute("href");
alert(href);
return true;
}
if (window.addEventListener) {
window.addEventListener("mouseover", handler, false);
} else if (window.attachEvent) {
document.body.attachEvent("onmouseover", handler);
} else {
window.onmouseover = handler;
}
})();