[Webkit-unassigned] [Bug 23477] Support for WCSS extensions

[bugzilla-daemon at webkit.org]

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


hyatt at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #26997|review?(darin at apple.com)    |review-
               Flag|                            |




------- Comment #5 from hyatt at apple.com  2009-01-29 10:11 PDT -------
(From update of attachment 26997)
Looks pretty good.  Here are some comments:

Your handling of the display property is not correct.  You can't mutate
overflowX at that time or you won't pick up all cases.  Consider the example of
"display:inherit", where someone inherits a display type of -wap-marquee.  Your
code won't set overflowX in that case.  A better place for that code would be
in adjustRenderStyle in CSSStyleSelector.  That code runs after the style has
been fully determined, so at that time you can ask if the display type is
-wap-marquee and then set overflowX to OMARQUEE.

I know you probably plan to implement them, but for this patch please remove
the CSS properties from CSSPropertyNames.in that aren't being handled yet.

You need to patch CSSComputedStyleDeclaration to handle all of these new
properties, so check that out.


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