[Webkit-unassigned] [Bug 20032] CSS 'content' attribute does not support 'none' or 'normal' values

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=20032


Darin Adler <darin at apple.com> changed:

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




--- Comment #6 from Darin Adler <darin at apple.com>  2009-08-20 06:14:59 PDT ---
(From update of attachment 35196)
> +            if (equalIgnoringCase(val->string, "none") || equalIgnoringCase(val->string, "normal")) {
> +                parsedValue = CSSPrimitiveValue::create("", CSSPrimitiveValue::CSS_STRING);
> +            }

There should not be braces here.

Changing "content: none" and "content: normal" into "content: ''" at the parser
level is wrong because it prevents us from doing the right thing with computed
style. To make computed style work properly we need to know the actual value of
the property. See bug 23668 for something tracking the need to implement
computed style for the content attribute.

And I do not think it is correct to treat "content: normal" exactly the same as
"content: ''". At the moment it mostly works only because the code in
RenderObject::createObject completely ignores the content property if it's not
an image.

But if you test this with the <br>, <frameset>, and <image> elements, I think
you'll see that it's not working correctly. All three of those classes do
something different if there is contentData than if there is not. I believe
that "content: normal" should not change things.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.