<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Nov 25, 2009, at 10:38 AM, Darin Fisher wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Wed, Nov 25, 2009 at 10:35 AM, Dan Bernstein <span dir="ltr">&lt;<a href="mailto:mitz@apple.com">mitz@apple.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
On Nov 25, 2009, at 9:55 AM, Darin Fisher wrote:<br>
<br>
&gt; Why are some layout tests renamed with a suffix of -disabled? &nbsp;Why aren't such tests simply added to the skipped list? &nbsp;Is -disabled simply the old way?<br>
<br>
</div>Usually a test is disabled, with a bug filed to re-enable it, when a WebKit bug makes it impossible to run the test (e.g. it crashes DumpRenderTree) or makes the test produce different results on each run (this can also be a bug in the test). The skipped lists are platform-specific, so they are not a good way to deal with such situations.</blockquote>
</div><br><div>My concern is that some tests may pass on some ports but not others.</div></blockquote><div><br></div><div>For such cases, the skipped list is the way to go.</div><br><blockquote type="cite"><div>If the tests are -disabled, then it prevents them from being run on ports where the tests function properly.</div></blockquote><div><br></div><div>I see now that I didn’t make myself clear. By “WebKit bug” above I meant a bug that affects all ports and platforms (maybe I should have said “a core WebKit bug” or “a WebCore bug” :-) ).</div><br><blockquote type="cite"><div>Since skipped lists prevent the test from being executed, doesn't it solve the problem of disabling a test for ports that can't handle it? &nbsp;What am I missing? :-)</div></blockquote><div><br></div><div>The “tests that crash/hang/behave unpredictably on all ports” part.</div><div><br></div><div>There aren’t many -disabled tests, and I think it’s a good idea to audit them and make sure that (a) they can’t be re-enabled (b) they can’t just be skipped on some platforms and run on others and (c) there’s a bug filed on re-enabling them (or fixing the reason why they can’t be enabled).</div></div></body></html>