[Webkit-unassigned] [Bug 60249] Make RenderStyle::setColumnBreakInside() reject unsupported enum values.

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Luke Macpherson <macpherson at chromium.org>  2011-05-04 23:42:04 PST ---
(In reply to comment #2)
> (From update of attachment 92375 [details])
> I'm confused why we'd want this change.  It seems render style is a dumb storage class and CSSStyleSelector shoudl be where all the mapping logic is.

See:
http://www.w3.org/TR/css3-multicol/#break-before-break-after-break-inside

It boils down to "always" is never a valid thing to set for this property. It is only in the enum because the enum is being shared with break-before and break-after.

The reason I want to remove it from CSSStyleSelector is that I am trying to make that code as homogeneous as possible so that we can have as much code sharing as possible when the property handling is moved into CSSStyleApplyProperty.

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