WebKitTestRunner needs layoutTestController.queueLoadingScript
<rdar://problem/8213870>
Created attachment 169379 [details] patch
Comment on attachment 169379 [details] patch Attachment 169379 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/14390918
Created attachment 169394 [details] patch v2 should satisfy MAC compiler
Comment on attachment 169394 [details] patch v2 View in context: https://bugs.webkit.org/attachment.cgi?id=169394&action=review > Tools/WebKitTestRunner/WorkQueueManager.cpp:169 > + enqueue(new ScriptItem</*loading*/ true>(script)); Wouldnt this be nicer with an enum?
Created attachment 169838 [details] patch v3 Introduced enum as work queue operation result. Thanks, Kenneth
Comment on attachment 169838 [details] patch v3 View in context: https://bugs.webkit.org/attachment.cgi?id=169838&action=review > Tools/WebKitTestRunner/WorkQueueManager.cpp:61 > + NotLoading NonLoading? or DisallowLoading, AllowLoading?
(In reply to comment #7) > (From update of attachment 169838 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=169838&action=review > > > Tools/WebKitTestRunner/WorkQueueManager.cpp:61 > > + NotLoading > > NonLoading? or DisallowLoading, AllowLoading? Think NonLoading is better
Created attachment 169846 [details] to be landed NotLoading -> NonLoading
Comment on attachment 169846 [details] to be landed Clearing flags on attachment: 169846 Committed r132041: <http://trac.webkit.org/changeset/132041>
All reviewed patches have been landed. Closing bug.
*** Bug 98141 has been marked as a duplicate of this bug. ***