[webkit-reviews] review denied: [Bug 35944] Add an appengine app to host chromium layout test results : [Attachment 50351] Fixed a CR in proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Mar 12 08:49:52 PST 2010


Adam Barth <abarth at webkit.org> has denied Victor Wang <victorw at chromium.org>'s
request for review:
Bug 35944: Add an appengine app to host chromium layout test results
https://bugs.webkit.org/show_bug.cgi?id=35944

Attachment 50351: Fixed a CR in proposed patch
https://bugs.webkit.org/attachment.cgi?id=50351&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
This should go in a subdirectory of WebKitTools, analogous to
QueueStatusServer, which is another of our AppEngine apps.

I'm not sure we want svn:executable or svn:eol-style set on these files.

+ class DeleteFile

This app looks wide open for anonymous folks on the interest to screw with out
datastore.  Do we care?

Do we want some CSS to make this pretty?  We can do that in another patch.


More information about the webkit-reviews mailing list