Более корректнее было бы сделать так:
AttrSet = function(element,attr){
if (attr.href){
element.setAttributeNS(SVG.xlink,'href',attr.href);
delete attr.href;
}
for (var i in attr){
element.setAttribute(i,attr[i]);
}
}
Нет нужды проверять каждый атрибут на соответствие 'href', поскольку у одного элемента не может быть двух одинаковых атрибутов.