Более корректнее было бы сделать так:
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', поскольку у одного элемента не может быть двух одинаковых атрибутов.