<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On May 3, 2011, at 4:01 PM, Ojan Vafai wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">The results load considerably faster now. For runs where many tests fail, the filesize is considerably smaller than the old-run-webkit-tests <meta charset="utf-8">filesize. Also, I've added in the image toggling behavior of old-run-webkit-tests and made the aesthetics a bit closer to the old-run-webkit-test format.<div>

<div><br></div><div>Some rough file sizes if you care about the details.</div><div><br></div><div>old-run-webkit-tests 5 failures: 5k<br><meta charset="utf-8">old-run-webkit-tests 500 failures: 250k<br><meta charset="utf-8">new-run-webkit-tests 5 failures: 1k json + 25k htmljs<br>

<meta charset="utf-8">new-run-webkit-tests 500 failures: 60k json + 25k html/js<br></div></div></blockquote><div><br></div>Thanks for all the work you've put into this, Ojan! Is there a link we can use to look at the latest and greatest?</div><div><br></div><div>-Adam</div><div><br><blockquote type="cite"><div><div><div class="gmail_quote">On Fri, Apr 22, 2011 at 11:04 AM, Ojan Vafai <span dir="ltr"><<a href="mailto:ojan@chromium.org">ojan@chromium.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="gmail_quote"><div class="im">On Thu, Apr 21, 2011 at 10:05 PM, Maciej Stachowiak <span dir="ltr"><<a href="mailto:mjs@apple.com" target="_blank">mjs@apple.com</a>></span> wrote:<br>

</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div style="word-wrap:break-word"><div><div><div class="im"><div>On Apr 21, 2011, at 4:47 PM, Ojan Vafai wrote:</div></div><div class="im"><blockquote type="cite"><div class="gmail_quote">On Thu, Apr 21, 2011 at 4:21 PM, Maciej Stachowiak <span dir="ltr"><<a href="mailto:mjs@apple.com" target="_blank">mjs@apple.com</a>></span> wrote:</div>



</blockquote></div></div><div class="im"><div><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div style="word-wrap:break-word"><div>(2) Slow to load (apparently it loads a 3 meg JSON file before displaying anything?)</div></div></blockquote><div><br></div><div>Loading it locally is very fast for me. I guess I should try it on a laptop though.</div>



</div></blockquote><div><br></div></div><div>I'm on a laptop. On my home WiFi, it takes 5-15 seconds.</div></div></div></div></blockquote><div><br></div><div>Yikes.</div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div style="word-wrap:break-word"><div><div><blockquote type="cite"><div class="gmail_quote"><div>For context, we reuse the JSON file that's used for the test runtime treemaps, which means we need an entry for each test. That json will will get .5 meg smaller shortly. There are a number of approaches to make this smaller if needed. Some options would be to generate a separate JSON file, or to make the data format in the existing json file more compact.</div>



</div></blockquote><div>It seems to me that only data on unexpected failures should be needed for initial display. Making the page load fast seems more important than convenience of reusing an existing JSON file. I hope you will agree that 5-15 second load time is not acceptable. I'm sure it would be even worse in poor network conditions, where even the old-style pages can be a challenge to load. I would also like to be able to look at results pages on my iPhone or iPad without invoking the OOM killer.</div>



</div></div></div></blockquote><div><br></div></div><div>This should be relatively straightforward to fix. I'll ping again in a day or two when it's done. I expect we should be able to get comparable, if not smaller size than the old style pages.</div>

<div class="im">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div style="word-wrap:break-word"><div>(3) I like PrettyPatch format better than wdiff format.</div></div>

</blockquote><div><br></div><div>This is orthogonal. new-run-webkit-tests will use whichever one is available on the system. The results.html file will display whichever one was used.</div></div></blockquote><div>What would the bots display if this is deployed?</div>



</div></div></div></blockquote><div><br></div></div><div>I'm not 100% sure. Dirk can confirm, but I believe it will use whichever one is on the system. So, if the bot has pretty-patch, but not wdiff, it would only show the pretty-patch link. The Chromium bot I linked to has wdiff, but not pretty-patch, hence the wdiff links.</div>



<div><br></div><font color="#888888"><div>Ojan </div></font></div>
</blockquote></div><br></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>