Commit Graph

67 Commits

Author SHA1 Message Date
Garrett Robinson
94ae55944b Bug 875126 - Make cgdb an option for mach --debugger. r=ted 2013-06-12 16:17:11 -04:00
Nathan Froyd
b5b1e03424 Bug 875332 - add cgdb to the known debuggers list; r=jmaher 2013-05-23 10:56:18 -04:00
Nathan Froyd
a7e95eedb6 Bug 873493 - add lldb to the list of known debuggers; r=jmaher 2013-05-17 11:22:00 -04:00
Joel Maher
a849266f5f Bug 827446 - update mochitest, reftest, xpcshell to use mozcrash. r=jhammel 2013-03-21 09:19:34 -04:00
Ed Morley
e978d29cf3 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
734e31bc68 Bug 850681 - Part 10: Tweak comments; r=ted 2013-03-18 17:18:08 +00:00
Ed Morley
80cc2c540c 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
5712f4cef3 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
923f59d03d 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
09a9fa4aec 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
fff4f9ec3d Bug 850681 - Part 5: Clean up logging; r=ted 2013-03-18 17:18:05 +00:00
Ed Morley
ed94e8b2c2 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
490b7cff8c 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
f93c9bb039 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
0278697df2 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
faac840dc7 Backed out changeset 1c19d2a03d90 (bug 827446) for packaging bustage on a CLOSED TREE. 2013-03-13 09:22:00 -04:00
Joel Maher
5256123c16 Bug 827446 - update mochitest, reftest, xpcshell to use mozcrash. r=jhammel 2013-03-13 07:38:00 -04:00
Ed Morley
e0d39230fc 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
e00f8d635f Bug 782633 - Followup to fix exception; r=me 2013-01-12 18:36:54 +00:00
Ed Morley
a10319d661 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
618d19c490 Bug 817349 - Ensure topFrame is always defined; r=ted 2012-12-03 14:30:35 +00:00
Ed Morley
5d31b56d86 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
178b788fed 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
cfdb5270d1 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
fe94b3e6fd 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
ba79bd8b4b Bug 728294 - Part 1 - Remove old debug log parser; r=ted 2012-08-03 12:36:59 +02:00
Ed Morley
b234075bef 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
a99e197f24 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
944476b0ab Bug 767896 - disable browser_pageInfo.js
--HG--
extra : rebase_source : 315e8b6c151c6c2c23fd2b3275addc7178dd36c2
2012-07-15 17:04:07 +02:00
Ed Morley
031cf9d430 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
ca171eec44 Bug 716478 - update licence to MPL 2. 2012-05-21 12:12:37 +01:00
Dão Gottwald
5f38e25ec2 Bug 734172 - Stop blaming tests that replace the first tab for leaking. r=ttaubert 2012-05-17 16:41:08 +02:00
Tim Taubert
658ca72f8a 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
5cf39d2bfa Bug 658738 - reduce MAX_LEAK_COUNT to match the status quo 2012-05-11 12:18:50 +02:00
Dão Gottwald
90c264a25a Bug 752467 - disable leaky tests. r=zpao 2012-05-10 20:09:25 +02:00
Phil Ringnalda
7f9cae1506 Merge m-c to m-i 2012-05-04 19:49:34 -07:00
Tim Taubert
7a5fb0d3d7 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
04e88ebd8a Bug 749361 - reduce MAX_LEAK_COUNT to match the status quo. r=ttaubert 2012-04-27 03:45:49 +02:00
Dão Gottwald
9f4754b67b 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
69e1e7dd64 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
24c15964f4 No bug - reduce MAX_LEAK_COUNT to match the status quo 2012-03-09 11:39:11 +01:00
Dão Gottwald
f5f9c22090 No bug - reduce MAX_LEAK_COUNT to match the status quo 2012-03-07 11:59:58 +01:00
Tim Taubert
2906a4045d Bug 730773 - Track shutdown leaks when DOMWindows *or* DocShells leaked (not and); r=ctalbert 2012-03-02 12:20:20 +01:00
Tim Taubert
39e83d0057 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
685437b892 Bug 703768 - Don't confuse Windows paths with URLs. r=ted 2011-11-29 14:13:08 -08:00
William Lachance
a3bbb32ede 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