Bug 18799 - apl.aip.org menu does not work (property names should be case sensitive)
Summary: apl.aip.org menu does not work (property names should be case sensitive)
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Darin Adler
URL: http://apl.aip.org/
Keywords: HasReduction, InRadar
Depends on:
Reported: 2008-04-29 03:52 PDT by Michael Zedler
Modified: 2008-06-02 10:32 PDT (History)
4 users (show)

See Also:

reduction (587 bytes, text/html)
2008-04-29 10:33 PDT, Marcus
no flags Details
patch (14.69 KB, patch)
2008-05-30 09:06 PDT, Darin Adler
sullivan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Zedler 2008-04-29 03:52:53 PDT
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.
Comment 1 Marcus 2008-04-29 10:33:36 PDT
Created attachment 20890 [details]

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.
Comment 2 David Kilzer (:ddkilzer) 2008-04-30 09:20:05 PDT
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.

Comment 3 David Kilzer (:ddkilzer) 2008-04-30 09:20:33 PDT
Comment 4 Darin Adler 2008-05-21 12:31:46 PDT
The bug seems to be in JSCSSStyleDeclarationCustom.cpp in cssPropertyName.

The fix is in this line:


If we just remove the toASCIILower, that would fix the bug.
Comment 5 Darin Adler 2008-05-30 09:06:59 PDT
Created attachment 21433 [details]
Comment 6 Darin Adler 2008-06-02 10:32:24 PDT
Committed revision 34312.