[webkit-dev] Please don't leave entries for rebaseline in TestExpectation files

Ryosuke Niwa rniwa at webkit.org
Fri Mar 22 14:07:58 PDT 2013

On Fri, Mar 22, 2013 at 1:12 AM, Ryosuke Niwa <rniwa at webkit.org> wrote:

> On Thu, Mar 21, 2013 at 1:57 AM, Ryosuke Niwa <rniwa at webkit.org> wrote:
>> On Thu, Mar 21, 2013 at 1:31 AM, Robert Hogan <lists at roberthogan.net>wrote:
>>> On Thursday, 21 March 2013, Ryosuke Niwa wrote:
>>>> I used to pull results from the bots where possible but creating
>>>>> inconsistency between png/text results is not good.
>>>> It is unfortunate but it's much better than losing the complete test
>>>> coverage.
>>> If that's the case then I'm happy to land whatever garden-o-matic pulls
>>> in or I can sweep from the bots, even if it means that png results for Mac,
>>> Qt, et al. go bad as a result.
>>> I guess we will always have ports whose bots do not run pixel tests so
>>> if those ports are happy to live with the downsides of doing that then
>>> there really is no obstacle to authors owning the job of updating the
>>> baselines for all ports when they land a change.
>>> IMHO ports who don't run pixel tests would be better off deleting any
>>> png results they have in the tree. Is there a reason Mac hasn't done that?
>>> Don't you get lots of failures when you run pixel tests locally?
>> Yes, but I'd argue that it's better than losing the test coverage.
>> By the way, we can easily address this problem by always generating pixel
>> results for unexpectedly failing tests. Namely, we can force --pixel when
>> we're retrying tests.
> I've posted a patch on https://bugs.webkit.org/show_bug.cgi?id=112898 to
> implement this. Once that patch is landed, you can grab pixel results from
> any EWS bots and bots on build.webkit.org.

Landed it in http://trac.webkit.org/changeset/146657. Once EWS bots catch
up, we can start grabbing both text and pixel results off of Chromium and
Mac EWS bots.

- R. Niwa
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20130322/096c54ff/attachment.html>

More information about the webkit-dev mailing list