Bug 6619 – Affects delta.com: getting and setting cssText property not supported for rules (only style decls)

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Geoffrey Garen]

This bug is also in Radar as <rdar://3486985>

Getting and setting cssText does not work with rules, even though I recently made it work for style 
declarations. I don't have a test case for this yet, but am filing the bug to remind myself.

-------------------------------------------

<GMT02-Aug-2005 00:15:03GMT> Beth Dakin:
<rdar://problem/4199876> indicates a problem with the rendering of http://www.delta.com

The problem appears to be that they try to set cssText. It is failing at the following line in one of the 
Javascript files:

if(document.styleSheets[i].cssRules[j].cssText.indexOf("img") != -1)

And consequently does not trim the repeated images. I have attached a reduction of delta.com....it can 
probably be reduced more, but this is clear enough to see the problem. I added two alert boxes 
too...only one will appear in Safari because it aborts the Javascript before the second. They will both 
appear -- twice, in fact -- in Firefox.

[Geoffrey Garen]

Created attachment 5743 [details]
Reduction.

[Anders Carlsson]

The cssText problem is not visible in ToT. However, sometimes getting CSS shorthand properties doesn't 
work. I've attached a fix for that in #6653

This does not fix the rendering issue though. I've come up with a reduction that fails to render correctly in 
Firefox and Safari but works in WinIE.

[Anders Carlsson]

Created attachment 5762 [details]
Better reduction

[Beth Dakin]

Created attachment 6829 [details]
Yet another reduction

Here is a slightly simpler reduction. WinIE ignores the 6 pixels on padding and pink div cuts off right at the bottom of the image. Safari and Firefox honor the padding. I am not sure why WinIE knows to ignore it. Also, if you make a simple adjustment, like remove the font-size style in the css, then WinIE honors the padding too, and the pink div sticks out the bottom. Crazay!

[Maciej Stachowiak]

Closing, since the site has changed to not longer have this bug, and we don't want to make a change to work with the old site, because that would require breaking the CSS box model.

[Alice Liu]

removing hitlist keyword

[Alice Liu]

whoops, putting it back for tracking purposes.