[webkit-reviews] review granted: [Bug 86152] The order of CSS properties is wrong in background shorthand : [Attachment 142865] Patch-fixes-layout-test

[bugzilla-daemon at webkit.org]

Ryosuke Niwa <rniwa at webkit.org> has granted Joe Thomas
<joethomas at motorola.com>'s request for review:
Bug 86152: The order of CSS properties is wrong in background shorthand
https://bugs.webkit.org/show_bug.cgi?id=86152

Attachment 142865: Patch-fixes-layout-test
https://bugs.webkit.org/attachment.cgi?id=142865&action=review

------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=142865&action=review


>>> Source/WebCore/css/StylePropertyShorthand.cpp:39
>>> +	     CSSPropertyBackgroundClip,
>> 
>> The said specification doesn't seem to say anything about origin and clip.
> 
> Paraphrasing from the spec http://www.w3.org/TR/css3-background/#background:
> 
> If one <box> value is present then it sets both ‘background-origin’ and
‘background-clip’ to that value. If two values are present, then the first sets
‘background-origin’ and the second ‘background-clip’.

Ok.