[Webkit-unassigned] [Bug 86889] should we get rid of the SLOW modifier in test_expectations.txt?

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=86889





--- Comment #1 from Ryosuke Niwa <rniwa at webkit.org>  2012-05-18 13:33:16 PST ---
(In reply to comment #0)
> 3) Should it be a warning similar to "unexpected pass" if a test marked SLOW starts to complete quickly? Seems reasonable ...

I suspect that'll add a lot of noise. Most of slow tests I know of are not always slow.

> 4) Can we get away with only using SLOW for "SLOW PASS", and say that SLOW failures should not be allowed (since we want to move to a model where any test that fails for longer than a couple hours should get a new expectation/baseline checked in)?
> 
> 5) Can/should we just get rid of SLOW altogebther? If we did this, we would have to adjust the default timeout value for chromium upwards, and we would want to consider what the impact on bot cycle time should be

I think we should do that. But how many tests are marked slow today?  Maybe we can split them into smaller piececs to make them fast.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.