mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-12-01 17:23:59 +00:00
Bug 850681 - Part 3: Output all of the leaked object lines as TEST-INFO rather than a combination of TEST-INFO and TEST-UNEXPECTED-FAIL; r=ted
Since the main leak summary will soon include an overview of the leaked objects, we no longer need/want a subset of the individual leak lines showing up in TBPL's annotated summary (TBPL's parser matches on 'TEST-UNEXPECTED-FAIL').
This commit is contained in:
parent
481990d90a
commit
a5009bbcf3
@ -323,7 +323,6 @@ def processSingleLeakFile(leakLogFileName, PID, processType, leakThreshold):
|
||||
leaks = open(leakLogFileName, "r")
|
||||
crashedOnPurpose = False
|
||||
totalBytesLeaked = None
|
||||
numObjects = 0
|
||||
for line in leaks:
|
||||
if line.find("purposefully crash") > -1:
|
||||
crashedOnPurpose = True
|
||||
@ -349,16 +348,9 @@ def processSingleLeakFile(leakLogFileName, PID, processType, leakThreshold):
|
||||
else:
|
||||
instance = "instance"
|
||||
rest = ""
|
||||
numObjects += 1
|
||||
if numObjects > 5:
|
||||
# don't spam brief tinderbox logs with tons of leak output
|
||||
prefix = "TEST-INFO"
|
||||
else:
|
||||
prefix = "TEST-UNEXPECTED-FAIL"
|
||||
log.info("%(prefix)s %(process)s| leakcheck | leaked %(numLeaked)d %(instance)s of %(name)s "
|
||||
log.info("TEST-INFO %(process)s| leakcheck | leaked %(numLeaked)d %(instance)s of %(name)s "
|
||||
"with size %(size)s bytes%(rest)s" %
|
||||
{ "prefix": prefix,
|
||||
"process": processString,
|
||||
{ "process": processString,
|
||||
"numLeaked": numLeaked,
|
||||
"instance": instance,
|
||||
"name": name,
|
||||
|
Loading…
Reference in New Issue
Block a user