<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jun 28, 2012, at 11:57 PM, Horky Chen <<a href="mailto:horky@sina.com">horky@sina.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hi,<div><br></div><div>On Mac OS, if one time-out larger than 30s would be used, --time-out-ms cannot work well.</div><div><br></div><div>According to the run-webkit-tests script, custom Time-Out can be assigned for each test case. But, unfortunately, below line in LayoutTestControllerMac.mm blocked the setting if it is larger than 30s (waitUntilDone & notifyDone are used):</div><div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 13px/normal Ubuntu; "><span style="color: #3c5489">    static</span> <span style="color: #3c5489">const</span> <span style="color: #bd5a0c">CFTimeInterval</span> waitToDumpWatchdogInterval = <span style="color: #3c5489">30.0</span>;</div></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 13px/normal Ubuntu; "><br></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 13px/normal Ubuntu; "><span class="Apple-style-span" style="font-size: medium; ">That is hard code, and no parameter can be accepted to adjust it. </span>There are two time-out settings for one test case, is it possible to use common time-out setting? </div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 13px/normal Ubuntu; "><font class="Apple-style-span" size="3"><br></font></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 13px/normal Ubuntu; "><span class="Apple-style-span" style="font-size: medium; ">Would you please help to double check about it? </span></div></div></blockquote><div><br></div>Historically there has been absolutely no excuse to *expect* a single test take longer than 30 seconds.</div><div><br></div><div>Such a test needs to be redesigned or broken up in to smaller tests.</div><div><br></div><div>Thanks,</div><div>~Brady</div><div><br><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 13px/normal Ubuntu; "><span class="Apple-style-span" style="font-size: medium; "><br></span></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 13px/normal Ubuntu; "><span class="Apple-style-span" style="font-size: medium; ">Best Regards!</span></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 13px/normal Ubuntu; "><span class="Apple-style-span" style="font-size: medium; ">Horky</span></div></div>_______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev<br></blockquote></div><br></body></html>