2490b7e423d654ce5153d8171fc5d702faec55c2: Bug 827486, part 7 - add basic code gen test. r=bz
Andrew McCreight <amccreight@mozilla.com> - Mon, 18 Mar 2013 10:19:42 -0700 - rev 125268
Push 24838 by amccreight@mozilla.com at Mon, 18 Mar 2013 17:20:00 +0000
Bug 827486, part 7 - add basic code gen test. r=bz
07c51bdbd801eb09d2e3b8ebaf8b495bcc6ddd4e: Bug 827486, part 6 - generate headers and code for JS implemented WebIDL classes. r=bz
Andrew McCreight <amccreight@mozilla.com> - Mon, 18 Mar 2013 10:19:42 -0700 - rev 125267
Push 24838 by amccreight@mozilla.com at Mon, 18 Mar 2013 17:20:00 +0000
Bug 827486, part 6 - generate headers and code for JS implemented WebIDL classes. r=bz
9abdb274d5c8b9f23b6c839c8b4197b2ebb5cf01: Bug 827486, part 5: add the main JS impl code gen. r=bz
Andrew McCreight <amccreight@mozilla.com> - Mon, 18 Mar 2013 10:19:42 -0700 - rev 125266
Push 24838 by amccreight@mozilla.com at Mon, 18 Mar 2013 17:20:00 +0000
Bug 827486, part 5: add the main JS impl code gen. r=bz
b8e85d11e93dd32e6cd91e0add5c7011f3fff50b: Bug 827486, part 4: copy parts of example generator to JS impl generator. r=bz
Andrew McCreight <amccreight@mozilla.com> - Mon, 18 Mar 2013 10:19:41 -0700 - rev 125265
Push 24838 by amccreight@mozilla.com at Mon, 18 Mar 2013 17:20:00 +0000
Bug 827486, part 4: copy parts of example generator to JS impl generator. r=bz
8d31fbd561294dc8e4296b891c9a2aea4f59dc9a: Bug 827486, part 3 - add dependencies for CGBindingImplClass. r=bz
Andrew McCreight <amccreight@mozilla.com> - Mon, 18 Mar 2013 10:19:41 -0700 - rev 125264
Push 24838 by amccreight@mozilla.com at Mon, 18 Mar 2013 17:20:00 +0000
Bug 827486, part 3 - add dependencies for CGBindingImplClass. r=bz
32851c16ed452ebb75ca811ada632b35f2e9a059: Bug 827486, part 2 - Refactor example class codegen into common base class. r=bz
Andrew McCreight <amccreight@mozilla.com> - Mon, 18 Mar 2013 10:19:41 -0700 - rev 125263
Push 24838 by amccreight@mozilla.com at Mon, 18 Mar 2013 17:20:00 +0000
Bug 827486, part 2 - Refactor example class codegen into common base class. r=bz
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 125262
Push 24838 by amccreight@mozilla.com at Mon, 18 Mar 2013 17:20:00 +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 125261
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125260
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125259
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125258
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125257
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125256
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125255
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125254
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125253
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125252
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125251
Push 24837 by emorley@mozilla.com at Mon, 18 Mar 2013 17:18:44 +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 125250
Push 24836 by kgupta@mozilla.com at Mon, 18 Mar 2013 16:42:01 +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 125249
Push 24835 by rgiles@mozilla.com at Mon, 18 Mar 2013 16:35:00 +0000
Bug 851906 - Restore compatibility with older GStreamer versions. r=doublec
(0) -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip