[Webkit-unassigned] [Bug 45261] [NRWT] Add temp directory to all running drivers

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 7 15:28:46 PDT 2010


https://bugs.webkit.org/show_bug.cgi?id=45261





--- Comment #6 from Dirk Pranke <dpranke at chromium.org>  2010-09-07 15:28:46 PST ---
Hmm. It's not great that we're doing this work in the constructor and destructor; I've found that can lead to indeterminate results in general (although it's fine in this case).

There's actually a port.setup_test_run() method that would be the appropriate place to create the temp dir - it's called only when we're sure we actually want to start running tests, but before DumpRenderTree is ever launched. Unfortunately, there isn't a cleanup_test_run() method to match it.

I'll add both of those and refactor this (in a separate bug/change), unless there's an objection ...

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list