[webkit-reviews] review requested: [Bug 36245] Add option "target" to rebaseline_chromium_webkit_test.py : [Attachment 52088] Proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 30 15:16:47 PDT 2010


Victor Wang <victorw at chromium.org> has asked  for review:
Bug 36245: Add option "target" to rebaseline_chromium_webkit_test.py
https://bugs.webkit.org/show_bug.cgi?id=36245

Attachment 52088: Proposed patch
https://bugs.webkit.org/attachment.cgi?id=52088&action=review

------- Additional Comments from Victor Wang <victorw at chromium.org>
Updated check_build to re-use the image check port.

Eric,
Figuring out whether image diff exists and setting the target (if it is empty)
based
on the result is kind of rebaseline tool specific so I agree with Dirk here on
not having
these logics inside port object. let me know if you disagree and strongly
prefer
we should move these logics into port.


More information about the webkit-reviews mailing list