[Webkit-unassigned] [Bug 8351] display:none has no effect on <option> element

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=8351


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #8980|review?                     |review-
               Flag|                            |




------- Comment #6 from darin at apple.com  2006-06-23 21:04 PDT -------
(From update of attachment 8980)
Ideally this would not be done inside the renderer since the renderer is going
away, but I suppose Adele can just consider it when re-implementing it.

It seems very strange to check the style attached to an object directly without
doing any style resolution. If we are going to do it, we should not be
converting it to a string. We should be getting the CSSValue object and then
calling isPrimitiveValue(), casting to a CSSPrimitiveValue, calling getIdent()
on it, and then checking against CSS_VAL_NONE.

Besides not doing any style resolution, it also seems strange to single out
display:none and not implement any other properties, like visibility:hidden or
other stranger display values.

I'm going to review- this because at the very least we should use
getPropertyCSSValue instead of getPropertyValue.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.