<div dir="ltr">Thanks for the concrete examples, Dave! &nbsp;I tested all 3 of these, and haven&#39;t yet found any problems. &nbsp;But I don&#39;t have specific URLs. &nbsp;I also looked through the webkit bugs database as much as I could, and could not locate them.
<div><br></div><div>BTW - if the primary concern is not spinning the CPU, we could just drop the throttle down to 2 or 3ms, and the CPU will be largely idle on most machines. &nbsp;In a few years, with faster processors, we&#39;ll be able to drop to 1ms and still have largely idle CPU.</div>
<div><br></div><div>Would 3ms be viable in your eyes?</div>

<div><br></div><div>Mike</div><div><br><br><div class="gmail_quote">On Mon, Sep 29, 2008 at 8:58 PM, David Hyatt <span dir="ltr">&lt;<a href="mailto:hyatt@apple.com" target="_blank">hyatt@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 style="word-wrap:break-word">We encountered 100% CPU spins on <a href="http://amazon.com" target="_blank">amazon.com</a>, <a href="http://orbitz.com" target="_blank">orbitz.com</a>, <a href="http://mapquest.com" target="_blank">mapquest.com</a>, among others (looking through Radar histories). &nbsp;This was pre-clamp. &nbsp;Web sites make this mistake because they don&#39;t know any better, and it works fine in IE. &nbsp;It is a mistake these sites will continue to make, and Chrome is the only browser that will be susceptible. &nbsp;Being different from IE here is not a good thing. &nbsp;You will end up having to evangelize sites over and over to fix 100% CPU spins that occur only in your browser. &nbsp;Do you really want that kind of headache?<div>


<br></div><div>A new API will let Web apps get the performance they need while avoiding compatibility problems.</div><div><br></div><div>dave</div><div><br><div><div><div></div><div><div>On Sep 29, 2008, at 10:06 PM, Maciej Stachowiak wrote:</div>


<br></div></div><blockquote type="cite"><div><div></div><div><div style="word-wrap:break-word"><br><div><div>On Sep 29, 2008, at 7:26 PM, Mike Belshe wrote:</div><br><blockquote type="cite"><div dir="ltr">Hi,<div>
<br></div><div>One of the differences between Chrome and Safari is that Chrome sets the setTimeout clamp to 1ms as opposed to 10ms. &nbsp;This means that if the application writer requests a timer of less than 10ms, Chrome will allow it, whereas Safari will clamp the minimum timeout to 10ms. &nbsp;The reason we did this was to minimize browser delays when running graphical javascript applications.</div>


 <div><br></div><div>This has been a concern for some, so I wanted to bring it up here and get an open discussion going. &nbsp;My hope is to lower or remove the clamp over time.</div><div><br></div><div>To demonstrate the benefit, here is one test case which benefits from removing the setTimeout clamp. &nbsp;Chrome gets about a ~4x performance boost by reducing the setTimeout clamp. &nbsp;This programming pattern in javascript is very common.</div>


 <div><br></div><div><a href="http://www.belshe.com/test/sort/sort.html" target="_blank">&nbsp;&nbsp; http://www.belshe.com/test/sort/sort.html</a><br></div><div><br></div><div>One counter argument brought up is a claim that all other browsers use a 10ms clamp, and this might cause incompatibilities. &nbsp;However, it turns out that browsers already use widely varying values. </div>


</div></blockquote><div><br></div><div>I believe all major browsers (besides Chrome) have a minimum of either 10ms or 15.6ms. I don&#39;t think this is &quot;widely varying&quot;.</div><br><blockquote type="cite"><div dir="ltr">


<div>&nbsp;We also really haven&#39;t seen any incompatibilities due to this change. &nbsp;It is true that having a lower clamp can provide an easy way for web developers to accidentally spin the CPU, and we have seen one high-profile instance of this. &nbsp;But of course spinning the CPU can be done in javascript all by itself :-)</div>


</div></blockquote><div><br></div><div>The kinds of problems we are concerned about are of three forms:</div><div><br></div><div>1) Animations that run faster than intended by the author (it&#39;s true that 10ms vs 16ms floors will give slight differences in speed, but not nearly as much so as 10ms vs no delay).</div>


<div><br></div><div>2) Burning CPU and battery on pages where the author did not expect this to happen, and had not seen it on the browsers he or she has tested with.</div><div><br></div><div>3) Possibly slowing things dow if a page is using a 0-delay timer to poll for completion of network activity. The popular JavaScript library jQuery does this to detect when all stylesheets have loaded. Lack of clamping could actually slow down the loading it is intended to wait for.</div>


<div><br></div><div>4) Future content that is authored in one of Safari or Chrome that depends on timing of 0-delay timers will have different behavior in the other. Thus, we get less compatibility benefit for WebKit-based browsers through cross-testing.</div>


<div><br></div><div>The fact that you say you have seen one high-profile instance doesn&#39;t sound to me like there are no incompatibilities. It sounds like there are some, and you have encountered at least one of them. Points 1 and 2 are what made us add the timer minimum in the first place, as documented in WebKit&#39;s SVN history and ChangeLogs. We originally did not have one, and added it for compatibility with other browsers.</div>


<div><br></div><div>Currently Chrome gets an advantage on some benchmarks by accepting this compatibility risk. This leads to misleading performance comparisons, in much the same way as firing the &quot;load&quot; event before images are loaded would.</div>


<br><blockquote type="cite"><div dir="ltr"> <div><span>Here is a summary of the minimum timeout for existing browsers (you can test your browser with this page:&nbsp;<a href="http://www.belshe.com/test/timers.html" target="_blank">http://www.belshe.com/test/timers.html )</a></span></div>


 <div>Safari for the mac: &nbsp; 10ms</div><div>Safari for windows: &nbsp; &nbsp;15.6ms</div><div>Firefox: &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; 10ms or 15.6ms, depending on whether or not Flash is running on the system</div><div>IE : &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; 15.6ms</div>


 <div>Chrome: &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;1ms (future - remove the clamp?)</div><div><br></div><div>So here are a couple of options:<br>&nbsp;&nbsp; 1) Remove or lower the clamp so that javascript apps can run substantially faster.</div><div>


 &nbsp;&nbsp; 2) Keep the clamp and let them run slowly :-)</div> <div><br></div><div>Thoughts? &nbsp;It would be great to see Safari and Chrome use the same clamping values.</div></div></blockquote><div><br></div><div>Or there is option 3:</div>


<div><br></div><div>3) Restore the clamp for setTimeout and setInterval to 10ms for compatibility, and add a new setHighResTimer API that does not have any lower bound.&nbsp;</div><div><br></div><div>This would let aware Web applications get the same benefit, but without any of the compatibility risk to legacy Web content. The main argument against doing things this way is that it would add API surface area. But that seems like a small price to pay for removing the compatibility risk, and turning the change into something other browsers would be willing to adopt.</div>


<div><br></div><div>I would like to propose an API along these lines for HTML5 but I would prefer if we can achieve consensus in the WebKit community first.</div><div><br></div><div>Regards,</div><div>Maciej</div><div><br>


</div></div></div></div></div>_______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br><a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>


</blockquote></div><br></div></div></blockquote></div><br></div></div>