Implement css display property in CSSStyleApplyProperty.
Created attachment 117297 [details] Patch
Comment on attachment 117297 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117297&action=review > Source/WebCore/ChangeLog:3 > + Implement css display property in CSSStyleApplyProperty. css -> CSS > Source/WebCore/css/CSSStyleApplyProperty.cpp:1183 > + static inline bool isSVGElement(CSSStyleSelector* selector) I'm not a fan of this name, since the function answers more than just "is it an SVG element". > Source/WebCore/css/CSSStyleApplyProperty.cpp:1203 > + static void applyInitialValue(CSSStyleSelector* selector) > + { > + selector->style()->setDisplay(RenderStyle::initialDisplay()); > + } Shouldn't we have the if (isSVGElement.....) early return before setting the value in here, too?
Created attachment 118157 [details] Patch
Comment on attachment 118157 [details] Patch Clearing flags on attachment: 118157 Committed r102552: <http://trac.webkit.org/changeset/102552>
All reviewed patches have been landed. Closing bug.