1465173 - Automation output reports test harness pid instead of test pid

Status: RESOLVED FIXED

(Core :: JavaScript Engine, enhancement)

Description

[Steve Fink [:sfink] [:s:]]

This is a nuisance for msan tests, because currently the only way to associate a test with its detailed msan output is by pid, but the pids of the msan sanitize_log.NNNNN reports are of the actual test (appropriately enough), but that file doesn't specify what test ran. You need to correlate them with the log file, except it has the wrong pids so you can't.

Comment 1

[Steve Fink [:sfink] [:s:]]

Attachment: Set the pid correctly in the automation output

Implementation is messy. I kept it as an 'extra' thing because it doesn't make sense on Windows. That may have been a mistake; it might be cleaner to have it as a more first-class thing and just not set it on Windows. (Really, I should dig up what sort of equivalent thing exists on Windows, but at the moment I wouldn't make any use of it anyway.)

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7d0c7ab48964
Set the pid correctly in the automation output, r=nbp

Comment 3

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/7d0c7ab48964