[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 #9 from Dirk Pranke <dpranke at chromium.org>  2012-05-18 15:20:54 PST ---
(In reply to comment #8)
> (In reply to comment #7)
> > even the minis tend to run 2 or 4 threads in parallel.
> > 
> > It seems to me that we should be skipping any tests that are reliably timing out. I'm not sure what running them buys us, unless we are just hoping that they start working.
> 
> I've seen many tests that magically start working. It's typically because we fix some bug that was causing the test to timeout or breaks time race condition, etc...

Oh sure, that happens, but I'm not convinced that we should be running these tests every time just to hope that it starts happening. It seems to me that we should skip these and have the onus be on someone to fix them. could be that when someone actually goes to fix the bug, it turns out that the test is no longer failing.

I would probably say the same thing about tests that crash. Frankly, I'm disturbed that we have any tests that are marked as (just) CRASH at all - why haven't those bugs been fixed?

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