[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 #11 from Ojan Vafai <ojan at chromium.org>  2012-05-18 15:40:51 PST ---
To defend the status quo a bit:
1. If a test does output any text, we'll show that in the actual.txt file even if the test times out. So, this can be very helpful for diagnosing why a test is timing out and fixing it (esp for flaky timeouts that are hard to reproduce).
2. If a test crashes, we get a stacktrace. This makes it easier to quickly look at the output of a crashing test on the flakiness dashboard and see if you have an idea what's going wrong, how to fix it, or who to CC to get it fixed.
3. It's actually not that rare for tests to stop timing out or crashing. Many times the crash is not in the test code itself, but is just something like memory corruption, so it goes from one test to another as the tests we skip/run changes. If we skip them, then the list of tests we skip just keeps growing indefinitely.

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