e4a689014130eb6cee46fc7d991051a766b4af6f: Bug 827486, part 1: add JSImplementation extended attribute. r=bz
Andrew McCreight <amccreight@mozilla.com> - Mon, 18 Mar 2013 10:19:41 -0700 - rev 131693
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 827486, part 1: add JSImplementation extended attribute. r=bz
7ac3f76249e78b56fe64518d20b1c6787634f114: Bug 850681 - Part 11: Move processString to the message part of log lines & remove unhelpful process IDs; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:08 +0000 - rev 131692
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
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.
025f582f8005489ebeec78514eb9c7ef31da51a8: Bug 850681 - Part 10: Tweak comments; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:08 +0000 - rev 131691
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 850681 - Part 10: Tweak comments; r=ted
de7502c6b940eefd26148306c81caae5bae0e6eb: Bug 850681 - Part 9: Return early if possible during leak summary generation; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:07 +0000 - rev 131690
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
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.
ae92e0819f2b0e925436aba44318dd571083cfcb: Bug 850681 - Part 8: Only parse the leak log once; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:07 +0000 - rev 131689
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
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.
6d5de0363957979f5633b5eb0d935cac2e3fc4e5: Bug 850681 - Part 7: Use the cleaner |with open() as foo| pattern & and don't close/reopen the log unnecessarily; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:06 +0000 - rev 131688
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 850681 - Part 7: Use the cleaner |with open() as foo| pattern & and don't close/reopen the log unnecessarily; r=ted
321bca7cf97ace1e1e5db97d202e2335d0f7ad6f: Bug 850681 - Part 6: Clean up log parsing conditionals & move the leak threshold reminder to the calling function; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:06 +0000 - rev 131687
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
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.
2931467b2c54ec4b560a4d3a1e44239fc196a233: Bug 850681 - Part 5: Clean up logging; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:05 +0000 - rev 131686
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 850681 - Part 5: Clean up logging; r=ted
a40f95ea3951eb372d11a4ee944809fa551d4d57: Bug 850681 - Part 4: Add a partial list of leaked objects to the leak summary, to allow TBPL to make bug suggestions; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:05 +0000 - rev 131685
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
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, ...)
0c83c9036f9e70644541bafc7a34ee4074199b14: 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
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:04 +0000 - rev 131684
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
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').
c314aefc1244a6c370c163e0e29f5613eaebb77b: Bug 850681 - Part 2: Wait until the full leak log has been processed before outputting the leak total; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:04 +0000 - rev 131683
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
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.
102365bfe4d3344eef6d7ec451c1c8890f2a74fd: Bug 850681 - Part 1: Use the test name 'leakcheck' rather than 'automationutils.processLeakLog()'; r=ted
Ed Morley <emorley@mozilla.com> - Mon, 18 Mar 2013 17:18:03 +0000 - rev 131682
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
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.
22d16e3d3ea191e72c5e593d2e43061fcc97643e: Bug 852163 - Unregister pref observer on activity destroy so that it doesn't leak the activity. r=Cwiiis
Kartikaya Gupta <kgupta@mozilla.com> - Mon, 18 Mar 2013 17:41:36 +0100 - rev 131681
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 852163 - Unregister pref observer on activity destroy so that it doesn't leak the activity. r=Cwiiis
f7208a072eb75afc226616daf7b6b0a23da46f07: Bug 851906 - Restore compatibility with older GStreamer versions. r=doublec
Alessandro Decina <alessandro.d@gmail.com> - Sun, 17 Mar 2013 07:33:00 -0700 - rev 131680
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 851906 - Restore compatibility with older GStreamer versions. r=doublec
196693568165b81ee7df8c3ea6fce6c54338c036: Bug 851443 - Tests for richgrid/richgriditem bindings + stubMethod test helper. r=jimm
Sam Foster <sfoster@mozilla.com> - Mon, 18 Mar 2013 16:30:41 +0000 - rev 131679
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 851443 - Tests for richgrid/richgriditem bindings + stubMethod test helper. r=jimm
62512f057d58b3b5c4975ea3c51c43fafd15a7f5: Bug 845416 - Detect mangled package name in ANR reporter; r=blassey
Jim Chen <nchen@mozilla.com> - Mon, 18 Mar 2013 12:27:40 -0400 - rev 131678
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 845416 - Detect mangled package name in ANR reporter; r=blassey
16f7a494895be1a7d4462572236ebba68df8a75d: Bug 845416 - Limit ANR stack to Fennec stack; r=blassey
Jim Chen <nchen@mozilla.com> - Mon, 18 Mar 2013 12:27:40 -0400 - rev 131677
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 845416 - Limit ANR stack to Fennec stack; r=blassey
ee2e53392df1146327fdf5ea889df45a23333b99: Bug 851996, Simplify MessageEvent's jsval handling, r=mccr8
Olli Pettay <Olli.Pettay@helsinki.fi> - Mon, 18 Mar 2013 16:50:37 +0100 - rev 131676
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Bug 851996, Simplify MessageEvent's jsval handling, r=mccr8
867475abb8948a8e585f41f973ea1410968c93b2: merge backout
Dão Gottwald <dao@mozilla.com> - Mon, 18 Mar 2013 16:46:39 +0100 - rev 131675
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
merge backout
6678ea192a6c07dc9675151a6b94a3f21920e6f4: Backed out changeset f029edccac09
Dão Gottwald <dao@mozilla.com> - Mon, 18 Mar 2013 16:46:17 +0100 - rev 131674
Push 3582 by bbajaj@mozilla.com at Mon, 01 Apr 2013 20:50:56 +0000
Backed out changeset f029edccac09
(0) -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip