[Webkit-unassigned] [Bug 85267] NRWT with --leaks shows leaks from old runs

[bugzilla-daemon at webkit.org]

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


Dirk Pranke <dpranke at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aroben at webkit.org,
                   |                            |eric at webkit.org




--- Comment #2 from Dirk Pranke <dpranke at chromium.org>  2012-05-01 11:54:07 PST ---
Hm. I didn't write this code, so I'm cc'ing eric (it looks like this was mostly eric and adam's work). 

That said, I have a couple of thoughts. 

First, it looks like the leak files are written to the results directory, you could avoid this problem altogether by deleting any old leak files prior to the run either by running with --clobber-old-results or by doing a more specific deletion (e.g., in setup_test_run()).

Second, if you wanted to keep perhaps it's enough to look for any leak files written since the start of the test run? You could catch the start of the run in setup_test_run() or even Port.__init__() and then modify leaks_file_in_directory to take a timestamp and filter for files with a ctime (or an mtime) > that time.

I'd probably vote for option #2, above.

However, if you really wanted to go the uuid route, I would probably pass it through via the WorkerArguments class populated in manager.py and then passed to the worker in start_worker().

Then, of course, we could approach this a completely different way ...

Right now we check for leaks when we stop the process; the problem with capturing that output and passing it back to the manager is that it isn't directly correlated to a particular test, and so it doesn't fit the normal model of messaging the workers use. 

There's nothing to stop us from, for example, modifying driver.stop() to pass back an object and then sending that object back in a message to the manager, which could then aggregate the objects and write into a single file, as you suggest. It wouldn't be much different from the set_uuid() message you added, it would just go in the opposite direction.

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