SpriteMe (http://spriteme.org/) is a bookmarklet that uses jquery-1.3.2.js. A user reported a bug running SpriteMe on http://news.cnet.com/ in Safari on the Mac. The issue results from this code in jquery-1.3.2.js (from Sizzle):

if ( document.getElementsByClassName && document.documentElement.getElementsByClassName ) (function(){

var div = document.createElement("div");

div.innerHTML = "<div class='test e'></div><div class='test'></div>";

Opera can't find a second classname (in 9.6)

if ( div.getElementsByClassName("e").length === 0 )

return;

Safari caches class attributes, doesn't catch changes (in 3.2)

div.lastChild.className = "e";

if ( div.getElementsByClassName("e").length === 1 )

The error occurs when the last line is executed:

TypeError: Result of expression '$.element' [undefined] is not a function. i.i.com.com/cnwk.1d/html/rb/js/tron/oreo.moo.rb.combined.js:6

CNet uses mootools which redefines getElementsByClassName, and this causes the error. I can avoid the error by testing that getElementsByClassName is a native method:

if ( document.getElementsByClassName &&

-1!=document.getElementsByClassName.toString().indexOf("native code") &&

document.documentElement.getElementsByClassName ) (function(){

Is this a suitable fix? Should I be submitting this to Dojo?