[Webkit-unassigned] [Bug 86796] Change a bunch of things in the test_expectations.txt format

[bugzilla-daemon at webkit.org]

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





--- Comment #11 from Ojan Vafai <ojan at chromium.org>  2012-05-18 11:21:44 PST ---
(In reply to comment #10)
> (In reply to comment #9)
> > > Hm. I'm strongly tempted to kill SLOW instead of implement this :).
> > 
> > Please don't mix syntax changes with functionality changes in the same patch. If we want to change the meaning of anything, that should be in an isolated patch.
> 
> Of course. I wouldn't be likely to make all of these changes in one patch, either. Dunno, I am torn on SLOW ...

If we got rid of slow, we'd need to change a few other things to compensate.
1. We'd need to increase the timeout considerably.
2. We'd need to decrease the timeout for tests expected to timeout in order to maintain buildbot cycle times.

Part 2 has been the contentious part of getting rid of SLOW since it will then always be hard to tell if the test would still timeout if it weren't marked TIMEOUT. Alternately, we could just skip tests that are expected to reliably TIMEOUT and have the longer timeout fo tests that timeout inconsistently. I'd be happy with that.

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