RESOLVED FIXED 28292
Need to extend DumpRenderTree to expose number of worker threads
https://bugs.webkit.org/show_bug.cgi?id=28292
Summary Need to extend DumpRenderTree to expose number of worker threads
Andrew Wilson
Reported 2009-08-13 21:00:32 PDT
There's currently no way to track if/when worker threads actually shut down from javascript layout tests. We need to extend DumpRenderTree to expose this information so we can write layout tests to verify this functionality.
Attachments
proposed patch (43.12 KB, patch)
2009-08-13 21:45 PDT, Andrew Wilson
no flags
patch with fix for gtk type error (43.13 KB, patch)
2009-08-13 21:51 PDT, Andrew Wilson
eric: review+
atwilson: commit-queue-
Andrew Wilson
Comment 1 2009-08-13 21:45:54 PDT
Created attachment 34806 [details] proposed patch Turns out adding new APIs to WebKit is trickier than it appears. We need to put together a wiki page on this.
Andrew Wilson
Comment 2 2009-08-13 21:51:19 PDT
Created attachment 34808 [details] patch with fix for gtk type error
Eric Seidel (no email)
Comment 3 2009-08-17 17:37:48 PDT
Comment on attachment 34808 [details] patch with fix for gtk type error Wow. I'm just blown away by how thorough this change is. Really, wow. I don't think I would have had the stamina to change all 4 platforms like that. This is technically a style violation: 32 if (layoutTestController.workerThreadCount == 1) { 33 log("PASS: Worker thread created"); 34 } else { 35 log("FAIL: After thread creation: layoutTestController.workerThreadCount = " + layoutTestController.workerThreadCount); 36 } As far as I can tell, this all looks correct. I'm certainly no COM expert, so there could be errors there, but looks fine to me.
Andrew Wilson
Comment 4 2009-08-18 13:21:53 PDT
Committed as r47429.
Note You need to log in before you can comment on or make changes to this bug.