Commit Graph

64 Commits

Author SHA1 Message Date
Joel Maher
8caf21a371 Bug 827446 - update mochitest, reftest, xpcshell to use mozcrash. r=jhammel 2013-03-21 09:19:34 -04:00
Ed Morley
e1d2b651f0 Bug 850681 - Part 11: Move processString to the message part of log lines & remove unhelpful process IDs; r=ted
Our standard logging format is:
TEST-{INFO,UNEXPECTED-FAIL} | testName | message

For leak logs from processes other than the main process (eg plugin processes),
currently processSingleLeakFile outputs:
TEST-{INFO,UNEXPECTED-FAIL} | foo process 123 | testName | message

This patch turns these into:
TEST-{INFO,UNEXPECTED-FAIL} | testName | foo process: message

Note: The process ID is contained in the raw leak log (which gets printed to the
log anyway), so we don't need to repeat it in the main summary log lines. In
addition, the variable process ID often breaks TBPL bug suggestions, so should
be avoided where possible.
2013-03-18 17:18:08 +00:00
Ed Morley
186a466d7e Bug 850681 - Part 10: Tweak comments; r=ted 2013-03-18 17:18:08 +00:00
Ed Morley
74fad36dd3 Bug 850681 - Part 9: Return early if possible during leak summary generation; r=ted
Now that we close the log file before summary generation (rather than at the
end of processSingleLeakFile()), we can return early in some cases during
leak summary generation.
2013-03-18 17:18:07 +00:00
Ed Morley
6b1ebaa887 Bug 850681 - Part 8: Only parse the leak log once; r=ted
The existing code parses the leak log twice - first time to output the
raw log lines & and the second to output the analysis of them.

As much fun as it is to loop all the things (\o/), let's not.
2013-03-18 17:18:07 +00:00
Ed Morley
a24e6257f9 Bug 850681 - Part 7: Use the cleaner |with open() as foo| pattern & and don't close/reopen the log unnecessarily; r=ted 2013-03-18 17:18:06 +00:00
Ed Morley
8ad547d1ec Bug 850681 - Part 6: Clean up log parsing conditionals & move the leak threshold reminder to the calling function; r=ted
The same threshold is used for all leak logs processed from the test run, so
unnecessary to repeat for each.
2013-03-18 17:18:06 +00:00
Ed Morley
010822e186 Bug 850681 - Part 5: Clean up logging; r=ted 2013-03-18 17:18:05 +00:00
Ed Morley
341c5ad068 Bug 850681 - Part 4: Add a partial list of leaked objects to the leak summary, to allow TBPL to make bug suggestions; r=ted
Current summary line:
TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 216 bytes during test execution

With patch parts 1-4:
TEST-UNEXPECTED-FAIL | leakcheck | 180 bytes leaked (Mutex, ReentrantMonitor, nsRunnable, nsTArray_base, nsThread)

...or with ellipses if more than 5 objects:
TEST-UNEXPECTED-FAIL | leakcheck | 3184 bytes leaked (BackstagePass, CondVar, Mutex, ReentrantMonitor, SharedScriptableHelperForJSIID, ...)
2013-03-18 17:18:05 +00:00
Ed Morley
c7b70d6131 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').
2013-03-18 17:18:04 +00:00
Ed Morley
5285499813 Bug 850681 - Part 2: Wait until the full leak log has been processed before outputting the leak total; r=ted
Will allow the addition of an overview of the leaked objects, in part 4.
2013-03-18 17:18:04 +00:00
Ed Morley
b1f2e48cf7 Bug 850681 - Part 1: Use the test name 'leakcheck' rather than 'automationutils.processLeakLog()'; r=ted
Reduces the chance of line wrapping in the TBPL annotated summary, and makes
my eyes bleed less every time I see leak failures.
2013-03-18 17:18:03 +00:00
Ryan VanderMeulen
03435943f0 Backed out changeset 1c19d2a03d90 (bug 827446) for packaging bustage on a CLOSED TREE. 2013-03-13 09:22:00 -04:00
Joel Maher
c7fbc01dc5 Bug 827446 - update mochitest, reftest, xpcshell to use mozcrash. r=jhammel 2013-03-13 07:38:00 -04:00
Ed Morley
779c83ee18 Bug 813577 - Use try/finally in checkForCrashes() to ensure minidump temp directories do not linger; r=gps 2013-02-08 16:32:07 +00:00
Ed Morley
f95bfd738e Bug 782633 - Followup to fix exception; r=me 2013-01-12 18:36:54 +00:00
Ed Morley
b7728b3b49 Bug 782633 - Make the "missing output line for total leaks!" failure mode a warning rather than TEST-UNEXPECTED-FAIL; r=ted 2013-01-11 22:20:21 +00:00
Ed Morley
28dc7d0586 Bug 817349 - Ensure topFrame is always defined; r=ted 2012-12-03 14:30:35 +00:00
Ed Morley
ad4d3d1634 Bug 813650 - Part 2: checkForCrashes() should output the top frame of a crash in a TBPL-parsable format; r=ted 2012-12-01 17:02:29 +00:00
Ed Morley
d541b5c41a Bug 813650 - Part 1: Don't print minidump stackwalk output incrementally, so we can implement part 2 without changing log order; r=ted 2012-12-01 16:58:40 +00:00
Ed Morley
295652c85c Bug 808410 - checkForCrashes should return true for cases where we hit an exception during minidump stackwalk; r=jmaher 2012-11-20 15:24:28 +00:00
Jesse Ruderman
c7bd194ff7 Bug 789275 - Better error message when trying to process a crash report without symbols. r=wlach. DONTBUILD 2012-09-15 11:33:02 -07:00
Tim Taubert
21026ba295 Bug 728294 - Part 1 - Remove old debug log parser; r=ted 2012-08-03 12:36:59 +02:00
Ed Morley
677e3ca3ad Bug 778433 - Raise ShutdownLeakLogger's MAX_LEAK_COUNT to something less annoying, until bug 728294 lands; r=me DONTBUILD on a CLOSED TREE 2012-07-28 16:22:47 +01:00
Tim Taubert
12b169c107 Bug 728426 - Opening and then closing bookmarks sidebar keeps the bookmarksPanel.xul and/or history-panel.xul document alive; r=dao 2012-07-21 19:37:56 +02:00
Dão Gottwald
accaddbba3 Bug 767896 - disable browser_pageInfo.js
--HG--
extra : rebase_source : 315e8b6c151c6c2c23fd2b3275addc7178dd36c2
2012-07-15 17:04:07 +02:00
Ed Morley
723cb5aa7f Bug 772388 - Make ShutdownLeakLogger output the list of leaking tests in a format that TBPL can individually star; r=ttaubert 2012-07-10 13:19:48 +01:00
Gervase Markham
87620f5676 Bug 716478 - update licence to MPL 2. 2012-05-21 12:12:37 +01:00
Dão Gottwald
78604b5478 Bug 734172 - Stop blaming tests that replace the first tab for leaking. r=ttaubert 2012-05-17 16:41:08 +02:00
Tim Taubert
76ce6de376 Bug 753815 - Intermittent INFO | runtests.py | Received unexpected exception while running application; r=ted 2012-05-14 17:02:26 +02:00
Dão Gottwald
612cfe389e Bug 658738 - reduce MAX_LEAK_COUNT to match the status quo 2012-05-11 12:18:50 +02:00
Dão Gottwald
a8cffe0ce0 Bug 752467 - disable leaky tests. r=zpao 2012-05-10 20:09:25 +02:00
Phil Ringnalda
e265da4be6 Merge m-c to m-i 2012-05-04 19:49:34 -07:00
Tim Taubert
437b56f4c2 Bug 740242 - runtests.py | Received unexpected exception while running application 'NoneType' object has no attribute 'group'; r=ted 2012-05-04 16:21:24 +02:00
Dão Gottwald
8b63ac7049 Bug 749361 - reduce MAX_LEAK_COUNT to match the status quo. r=ttaubert 2012-04-27 03:45:49 +02:00
Dão Gottwald
54b97438d1 Bug 751334 - Redundant TabView.init call in restoreWindow leaks the browser window when the window closes before delayedStartup was called. r=ttaubert 2012-05-04 18:17:25 +02:00
Dão Gottwald
1d8be0f2df Bug 734554 - Increase MAX_LEAK_COUNT in order to reduce the random orange volume 2012-03-13 13:51:56 +01:00
Dão Gottwald
9d7c3fd293 No bug - reduce MAX_LEAK_COUNT to match the status quo 2012-03-09 11:39:11 +01:00
Dão Gottwald
af6780dac9 No bug - reduce MAX_LEAK_COUNT to match the status quo 2012-03-07 11:59:58 +01:00
Tim Taubert
0b0a0a10de Bug 730773 - Track shutdown leaks when DOMWindows *or* DocShells leaked (not and); r=ctalbert 2012-03-02 12:20:20 +01:00
Tim Taubert
2cfabea640 Bug 683953 - Part 1 - Parse log output and collect statistics about leaked DOMWindows and DocShells; r=ted 2012-02-25 12:48:05 +01:00
Jesse Ruderman
bc3da18dce Bug 703768 - Don't confuse Windows paths with URLs. r=ted 2011-11-29 14:13:08 -08:00
William Lachance
e968a4efed Bug 679759 - Drop MINIDUMP_STACKWALK_CGI support, let harness download symbols as needed; r=ted 2011-11-15 04:33:21 +00:00
Cameron McCormack
5f0489f00f Bug 642175 - Part 2: Allow mochitests to clean up plugin and IPC process crash dumps. r=ted 2011-06-21 12:11:50 +12:00
Jesse Ruderman
f4c7d46370 Bug 606389 - when minidump_stack fails, show its error messages and/or exit code. r=ted, a=npotb 2010-11-06 13:30:32 -07:00
Ted Mielczarek
a2bce37bbd bug 598507 - wrap test harness process execution with 'arch -arch i386' on OS X 10.5. r=catlee, a=blocking
--HG--
extra : rebase_source : 7212fa8332fe6853f6dedd6bd23dc937f4b9ec86
2010-09-23 12:19:31 -04:00
Ted Mielczarek
1c9c3c7171 bug 569965 - automationutils.processLeakLog() should learn not to TEST-UNEXPECTED-FAIL every single object line when there are lots. r=philor
--HG--
extra : rebase_source : 4ce122222020d522f742b7e77bfacafe8f7f90bc
2010-06-10 08:27:47 -04:00
Ted Mielczarek
09691f1ca3 bug 563745 - Make automationutils.py's minidump processing able to send minidumps to minidump-stackwalk-cgi. r=catlee
--HG--
extra : rebase_source : ebbf745e5e77c55437cb05071f49d421d08956c6
2010-05-06 08:06:09 -04:00
Joel Maher
e2fbedec3f Bug 493748 Refactor xpcshell test harness to enable it to work for remote devices r=ted (comment 14) p=jmaher Landing on a CLOSED TREE 2010-03-12 14:57:29 -08:00
Ted Mielczarek
fffbd88cf3 bug 540627 - provide an environment variable to save minidumps from test runs. r=bsmedberg
--HG--
extra : rebase_source : d53ac17a06eb938e1fcd9a59e409b3467c04fed9
2010-02-03 14:15:17 -05:00