.removeAttr don't works with select element's size attribute

[LaySoft]

Reproduction code: ​http://bogex.hu/select.php

Both tests tries to remove 'multiple' and 'size' attribute from select element. The first test doing this with javascript's own removeAttribute method, and second with jQuery's removeAttr method. The first test works under Firefox 4.0b10, but second not:

Error: uncaught exception: [Exception... "Index or size is negative or greater than the allowed amount" code: "1" nsresult: "0x80530001 (NS_ERROR_DOM_INDEX_SIZE_ERR)" location: "​http://code.jquery.com/jquery-1.5.min.js Line: 16"]

On all other browsers works both the tests.

[jitter]

​test case

[jitter]

Note to self related to #7981

[mounir.lamouri]

FWIW, there is a Gecko/Firefox bug open about this issue: ​https://bugzilla.mozilla.org/show_bug.cgi?id=640636 The issue might be fixed in a minor FF4 release. Unfortunately, I don't think it will be fixed for FF4 final.

[mounir.lamouri]

This has been fixed in Firefox's trunk. It will be fixed in Firefox 5 (ETA 3 months) and *might* be fixed in a minor FF4 release.

[anonymous]

low? this breaks sooo much in firefox 4.

The issue can not set elem.size to value is and is NOT null. This should help you genius' fix it. I would assume that if value == then value = null but have no idea what that might break having limited knowledge of the jquery internals.

if ( value === null ) {

if ( elem.nodeType === 1 ) {

elem.removeAttribute( name );

}

} else {

elem[ name ] = value;

}

[timmywil]

This has been fixed in the upcoming attrhooks in jQuery 1.6. ​http://jsfiddle.net/timmywil/bAtgQ/

[Rick Waldron]

#8647 is a duplicate of this ticket.

[mounir.lamouri]

FWIW, Firefox 4.0.1 will have the same behavior as Firefox 3.6, thus the jQuery fix will not be needed anymore.