apl.aip.org menu does not work (property names should be case sensitive)

• Component: CSS
• Status: RESOLVED FIXED

[Michael Zedler]

Hovering over the menu "General Information" on http://apl.aip.org/ should open submenus. Works in FF+Opera, fails in Safari 3.1.1 and ToT.

[Marcus]

Created attachment 20890 [details]
reduction

Code snippet from the linked page:
 
ZIndex = 5000;
with (object.style) { 
    zIndex = --ZIndex; 
} 

In WebKit the first letter of the property name inside the with-statement is not case sensitive.  ZIndex is the same as zIndex. The above code sets zIndex to -1 and the menu is hidden.

[David Kilzer (:ddkilzer)]

Confirmed with Safari 3.1.1 (5525.20) on Mac OS X 10.5.x (Leopard) and with WebKit nightly build r32698.

Works on Firefox 2.0.0.x.

[David Kilzer (:ddkilzer)]

<rdar://problem/5900484>

[Darin Adler]

The bug seems to be in JSCSSStyleDeclarationCustom.cpp in cssPropertyName.

The fix is in this line:

    name.append(toASCIILower(propertyName.data()[i++]));

If we just remove the toASCIILower, that would fix the bug.

[Darin Adler]

Created attachment 21433 [details]
patch

[Darin Adler]

Committed revision 34312.