2 つの理由で正しくありません。
・1 つの要素が同じ属性(ここでは onclick)を 2 つ以上持つことはできません。
・onclick 属性の中に javascript: を書く必要はありません(むしろ、書かないで下さい)。
注意事項として、
・JavaScript の文はセミコロン(;)で終わります。複数の文を並べられます。
・window.external は IE 独自拡張で、他のブラウザは対応していません。
<a href="#"
onclick="
if (window.external) window.external.AddFavorite('http://www.****.com','サイト名');
urchinTracker ('/bookmark');
return false;
">