[webkit-reviews] review denied: [Bug 17051] safari should treat "url()" as a valid CSS value : [Attachment 21531] another attempt to format patch properly

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has denied Bradley Meck <genisis329 at gmail.com>'s
request for review:
Bug 17051: safari should treat "url()" as a valid CSS value
http://bugs.webkit.org/show_bug.cgi?id=17051

Attachment 21531: another attempt to format patch properly
http://bugs.webkit.org/attachment.cgi?id=21531&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
WebCore/ChangeLog still doesn't include the URL of te bug.

WebKit/ChangeLog still has a gratuitous change in it (just a blank line, so
harmless).

LayoutTests/ChangeLog mentions the ChangeLog itself as one of the files
changed.

The LayoutTests patch includes the new test in its old path location,
LayoutTests/css2.1/t17051-url-empty.html, and doesn't include the expected
results for the new test.

I'm sorry -- I know you're trying hard to get this right but this is still not
ready to land.

review- because of the LayoutTests problems. The other problems are minor.