<div dir="ltr"><span style="line-height:normal">On Sat, Mar 7, 2015 at 9:02 PM, Brent Fulgham </span><span dir="ltr" style="line-height:normal">&lt;<a href="mailto:bfulgham@gmail.com" target="_blank">bfulgham@gmail.com</a>&gt;</span><span style="line-height:normal"> wrote:</span><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="auto"><div>Hi Antti,</div><div><br></div><div>Take a look at &lt;<a href="https://bugs.webkit.org/show_bug.cgi?id=142432" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=142432</a>&gt;. I haven&#39;t landed this yet, because I was hoping to get some kind of test for this. However, it seems to introduce no problems, so we could go ahead and land it.</div></div></blockquote><div><br></div><div>That was fast, thanks! Looks good as far as I can tell. Basic QOS support (thread priorities) would also be a useful addition at some point.  </div><div><span style="line-height:normal"><br></span></div><div><span style="line-height:normal">I don&#39;t think there is any specific test coverage (though WorkQueue is used heavily in WK2). At least a basic unit test under Tools/TestWebKitAPI/Tests/WTF would be nice.</span></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="auto"><div>Question: why do we have a separate WorkQueue implementation in DumpRenderTree *and* WebKitTestRunner? Seems like we could do some consolidation...</div></div></blockquote><div> </div><div>This might be because the code used to live in WK2 and was only recently moved to WTF. It would indeed be good to get rid of the duplication.</div><div><br></div><div><br></div><div>  antti</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="auto"><span class=""><font color="#888888"><div>-Brent<br><br></div></font></span><div><div class="h5"><div><br>On Mar 6, 2015, at 10:55 AM, Anders Carlsson &lt;<a href="mailto:andersca@apple.com" target="_blank">andersca@apple.com</a>&gt; wrote:<br><br></div><blockquote type="cite"><div><span>I think we used to have a WorkQueue implementation for Windows WK2 - anyone should be able to dig it up.</span><br><span></span><br><span>(Alternatively, we can just make an implementation using std::thread).</span><br><span></span><br><span>- Anders</span><br><span></span><br><blockquote type="cite"><span>On Mar 6, 2015, at 10:46 AM, Antti Koivisto &lt;<a href="mailto:koivisto@iki.fi" target="_blank">koivisto@iki.fi</a>&gt; wrote:</span><br></blockquote><blockquote type="cite"><span></span><br></blockquote><blockquote type="cite"><span>Hi,</span><br></blockquote><blockquote type="cite"><span></span><br></blockquote><blockquote type="cite"><span>I&#39;d like to start using WTF::WorkQueue abstraction in WebCore code. However we are currently missing a Windows implementation (WorkQueue used to be part of WK2). Anyone interested in making one? I assume it wouldn&#39;t be a difficult task for someone who knows the platform.</span><br></blockquote><blockquote type="cite"><span></span><br></blockquote><blockquote type="cite"><span></span><br></blockquote><blockquote type="cite"><span>   antti</span><br></blockquote><blockquote type="cite"><span>_______________________________________________</span><br></blockquote><blockquote type="cite"><span>webkit-dev mailing list</span><br></blockquote><blockquote type="cite"><span><a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a></span><br></blockquote><blockquote type="cite"><span><a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a></span><br></blockquote><span></span><br><span>_______________________________________________</span><br><span>webkit-dev mailing list</span><br><span><a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a></span><br><span><a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a></span><br></div></blockquote></div></div></div></blockquote></div><br></div></div>