[webkit-reviews] review granted: [Bug 36195] Add layout test flakiness (diagnostics) dashboard to TestResultServer appengine : [Attachment 51547] Correct app name and remove a few unused lines.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Mar 25 16:06:08 PDT 2010


Adam Barth <abarth at webkit.org> has granted Victor Wang <victorw at chromium.org>'s
request for review:
Bug 36195: Add layout test flakiness (diagnostics) dashboard to
TestResultServer appengine
https://bugs.webkit.org/show_bug.cgi?id=36195

Attachment 51547: Correct app name and remove a few unused lines.
https://bugs.webkit.org/attachment.cgi?id=51547&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
Thanks for addressing my feedback.  I'm not super excited about the dependency
on the chromium source server, but I don't see a better solution in the
meantime.

We have another AppEngine app in QueueStatusServer, which also doesn't have any
test coverage.	I'm deathly afraid of touching it for that reason.  A testing
framework would certainly be nice.  :)


More information about the webkit-reviews mailing list