[webkit-dev] TestExpectations syntax changes, last call (for a while, at least) ...
Elliot Poger
epoger at chromium.org
Thu Jun 14 13:51:08 PDT 2012
On Thu, Jun 14, 2012 at 4:44 PM, Peter Kasting <pkasting at chromium.org>wrote:
> On Thu, Jun 14, 2012 at 1:39 PM, Elliot Poger <epoger at chromium.org> wrote:
>
>> Can someone please remind me why IMAGE+TEXT even exists?
>>
>> Wouldn't it be simpler to just mark a test as follows?
>>
>> - IMAGE : allow image failure; go red if there is a text failure
>> - TEXT: allow text failure; go red if there is an image failure
>> - IMAGE TEXT: allow text and/or image failure
>>
>> The distinction is that IMAGE TEXT will allow image, text, or both to
> fail, thus making transitions among the three generate no events.
> IMAGE+TEXT says specifically that we expect both to fail and that if one
> starts passing, someone should do something. (For example, maybe someone
> checks in a partial rebaseline where they miss the image expectations.)
>
Thanks for the explanation.
That makes sense, although it seems to me that the problem of
no-events-generated-by-changes-in-actual-images-while-IMAGE-failure-is-expected
is about 100x worse for us.
But that's not a reason to hide these particular transitions! :-)
>
> PK
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20120614/d37fa5e4/attachment.html>
More information about the webkit-dev
mailing list