[webkit-reviews] review requested: [Bug 25489] background-position not shown : [Attachment 38503] A Fix

[bugzilla-daemon at webkit.org]

yaar at google.com has asked  for review:
Bug 25489: background-position not shown
https://bugs.webkit.org/show_bug.cgi?id=25489

Attachment 38503: A Fix
https://bugs.webkit.org/attachment.cgi?id=38503&action=review

------- Additional Comments from yaar at google.com
The patch depends on the patch for
https://bugs.webkit.org/show_bug.cgi?id=26541

CSS property "background-position" was not serialized into the shorthand string
value of "style.background". This is because position is stored into two
(non-standard?) properties background-position-x and background-position-y.
Since "style.backgroundPosition" does serialize as "position-x, position-y",
(e.g. 50% 50%) it made sense that the shorthand value of style.background will
do the same.

I fixed the layout test fast/dom/background-shorthand-csstext.html and
fast/dom/background-shorthand-expected.txt as the later was accepting a FAIL.