[webkit-reviews] review denied: [Bug 15713] Outside list bullets ignore text-align style : [Attachment 71562] Fix the position of outside list bullets

[bugzilla-daemon at webkit.org]

Andreas Kling <kling at webkit.org> has denied Renata Hodovan
<reni at inf.u-szeged.hu>'s request for review:
Bug 15713: Outside list bullets ignore text-align style
https://bugs.webkit.org/show_bug.cgi?id=15713

Attachment 71562: Fix the position of outside list bullets
https://bugs.webkit.org/attachment.cgi?id=71562&action=review

------- Additional Comments from Andreas Kling <kling at webkit.org>
I took this patch for a spin, and it broke the following tests on Snow Leopard:


css1/box_properties/border_left.html
css1/box_properties/margin_right.html
css1/box_properties/padding_left.html
css1/box_properties/padding_right.html
css2.1/t0805-c5521-brdr-l-02-e.html
editing/selection/extend-by-word-002.html
fast/css/background-shorthand-invalid-url.html
fast/css/empty-pseudo-class.html
fast/css/first-child-pseudo-class.html
fast/css/first-of-type-pseudo-class.html
fast/css/last-child-pseudo-class.html
fast/css/last-of-type-pseudo-class.html
fast/css/list-outline.html
fast/css/only-child-pseudo-class.html
fast/css/only-of-type-pseudo-class.html
fast/lists/001.html
fast/lists/002.html
fast/lists/003.html
fast/lists/007.html
fast/lists/008.html
fast/lists/ordered-list-with-no-ol-tag.html

Some of them are definitely progressions, but please look through them and
rebaseline as appropriate in the next patch iteration.

I uploaded the results here for your convenience:
http://chaos.troll.no/~kling/15713/layout-test-results/results.html