bb2538161269f1aabfa54342bf61602eebbec860: Bug 1064580 - Add definitions for onCurrentSelection to the listeners since RemoteFinder.jsm expects them to be present. Also add an error check in Finder.jsm for empty selections. r?evilpie draft
Jared Wein <jwein@mozilla.com> - Mon, 07 Mar 2016 16:37:34 -0500 - rev 337468
Push 12371 by jwein@mozilla.com at Mon, 07 Mar 2016 21:38:28 +0000
Bug 1064580 - Add definitions for onCurrentSelection to the listeners since RemoteFinder.jsm expects them to be present. Also add an error check in Finder.jsm for empty selections. r?evilpie MozReview-Commit-ID: E6lgH8OZhaH
f94507dbfd956c843cc5e98da538690582fcfee4: Bug 1254263 - Add a flag to allow disabling paint-skipping. r?mstange draft
Kartikaya Gupta <kgupta@mozilla.com> - Mon, 07 Mar 2016 16:25:00 -0500 - rev 337467
Push 12370 by kgupta@mozilla.com at Mon, 07 Mar 2016 21:25:29 +0000
Bug 1254263 - Add a flag to allow disabling paint-skipping. r?mstange MozReview-Commit-ID: 4bgvqEZcSWy
bfd4bfd80855c1f5dac69e552317ace95121136d: Bug 1236108: Add support for exception-context annotations for content processes to the crash reporter; r?bsmedberg draft
Aaron Klotz <aklotz@mozilla.com> - Mon, 07 Mar 2016 12:48:58 -0800 - rev 337466
Push 12369 by aklotz@mozilla.com at Mon, 07 Mar 2016 21:24:34 +0000
Bug 1236108: Add support for exception-context annotations for content processes to the crash reporter; r?bsmedberg This patch redefines XP_PATH_MAX on Windows to be MAX_PATH + 1. I did this because the longer definition would actually not work with most Windows APIs. Some APIs can work with longer lengths if the path is prefixed with "\\?\", but that is not guaranteed in general. See https://msdn.microsoft.com/en-us/library/windows/desktop/aa365247(v=vs.85).aspx#maxpath MozReview-Commit-ID: LhT5oM5hlkl
72e84a63f77d8745acfe37f9abd5c8a517b31ec4: Bug 1236108: Modify sandbox initialization code to use directory service to obtain content process temp directory; r?bobowen, haik draft
Aaron Klotz <aklotz@mozilla.com> - Mon, 07 Mar 2016 11:31:46 -0800 - rev 337465
Push 12369 by aklotz@mozilla.com at Mon, 07 Mar 2016 21:24:34 +0000
Bug 1236108: Modify sandbox initialization code to use directory service to obtain content process temp directory; r?bobowen, haik The previous patch in this series creates a new directory service entry specifically for obtaining the content process temp directory. This patch converts everything else to reference that entry. It also sets appropriate environment variables in the content processes so that system APIs automatically pick up the directory. This is necessary for the crash reporter to be able to call those APIs in exception handling contexts. MozReview-Commit-ID: DF6aNKrWnWp
542840808172a91cf205b5e02cb573bbb962c0e8: Bug 1236108: Add temp directory for sandboxed content processes to directory draft
Aaron Klotz <aklotz@mozilla.com> - Mon, 07 Mar 2016 11:26:35 -0800 - rev 337464
Push 12369 by aklotz@mozilla.com at Mon, 07 Mar 2016 21:24:34 +0000
Bug 1236108: Add temp directory for sandboxed content processes to directory service. This is needed so that chrome processes know where sandboxed content processes will be writing their temp files, and so that content processes know where to write; r?bsmedberg MozReview-Commit-ID: BK9bTxFGvZO
69d87838c4249d7b0d79aeb36bf5cc9e234202bd: Bug 1227367 - Test MarionetteTestRunner's test_handlers and testvars; r=automatedtester draft
Maja Frydrychowicz <mjzffr@gmail.com> - Thu, 03 Mar 2016 15:25:21 -0500 - rev 337463
Push 12368 by mjzffr@gmail.com at Mon, 07 Mar 2016 21:20:07 +0000
Bug 1227367 - Test MarionetteTestRunner's test_handlers and testvars; r=automatedtester BaseMarionetteTestRunner sets up an empty list of test_handlers; however it is essential for any child class have at least one test_handler, otherwise the harness is busted. So, I added a test to check that MarionetteTestRunner sets up the expected test_handlers. In doing so, I refactor BaseMarionetteTestRunner.__init__ to split out file i/o into other methods so I can mock them out, which involves modifying testvars code. So, I also added a test to make sure the parsing of testvars still works. MozReview-Commit-ID: GF5MKy8rN8C
8544182973b81e4033abd64478af59111b0e8d09: Bug 1227367 - Set logger fixture as a dependency for the kwargs fixture; r=automatedtester draft
Maja Frydrychowicz <mjzffr@gmail.com> - Tue, 01 Mar 2016 17:39:13 -0500 - rev 337462
Push 12368 by mjzffr@gmail.com at Mon, 07 Mar 2016 21:20:07 +0000
Bug 1227367 - Set logger fixture as a dependency for the kwargs fixture; r=automatedtester MozReview-Commit-ID: JwlktoYmyuG
74cc801a8b1b8c199e77c0aca1f9384ea756a17f: Bug 1227367 - Detect empty test suite in BaseMarionetteTestRunner; r=automatedtester draft
Maja Frydrychowicz <mjzffr@gmail.com> - Tue, 01 Mar 2016 17:37:16 -0500 - rev 337461
Push 12368 by mjzffr@gmail.com at Mon, 07 Mar 2016 21:20:07 +0000
Bug 1227367 - Detect empty test suite in BaseMarionetteTestRunner; r=automatedtester If the appropriate test_handler isn't available to the runner, tests specified at the command-line silently omitted. It's possible for the runner to create an empty test-suite. This patch adds asserts to detect an empty test-suite early and provide a more specific error message. MozReview-Commit-ID: 5XL4XZHCbMu
eba8329e66b721193fd989faa798e50e8f0e1baf: Bug 1227367 - Add tests for MarionetteHarness; r=automatedtester draft
Maja Frydrychowicz <mjzffr@gmail.com> - Tue, 01 Mar 2016 17:24:47 -0500 - rev 337460
Push 12368 by mjzffr@gmail.com at Mon, 07 Mar 2016 21:20:07 +0000
Bug 1227367 - Add tests for MarionetteHarness; r=automatedtester These tests aim to exercise the way MarionetteHarness is used in mach versus runtests.cli and downstream custom harnesses. Also refactor previous exit-code tests to use mock. MozReview-Commit-ID: 5baalNQz648
a96e2d484300c1495548a878563e2d6fcbdf5e1d: Bug 1227367 - Add checks to avoid AttributeErrors; r=automatedtester draft
Maja Frydrychowicz <mjzffr@gmail.com> - Fri, 12 Feb 2016 14:42:28 -0500 - rev 337459
Push 12368 by mjzffr@gmail.com at Mon, 07 Mar 2016 21:20:07 +0000
Bug 1227367 - Add checks to avoid AttributeErrors; r=automatedtester Depending on how MarionetteHarness is called, attributes like logger and pydebugger may not have been initialized. MozReview-Commit-ID: S2QzFnD1m4
135a9cd556f58dce73c7e667d4eb0c1c30cc1ca2: Bug 1227367 - Add TaskCluster harness unit test task draft
Maja Frydrychowicz <mjzffr@gmail.com> - Thu, 03 Mar 2016 15:24:16 -0500 - rev 337458
Push 12368 by mjzffr@gmail.com at Mon, 07 Mar 2016 21:20:07 +0000
Bug 1227367 - Add TaskCluster harness unit test task I will fix-up this commit to incorporate feedback and take advantage of the new "generic" TC tasks MozReview-Commit-ID: H08v8ow4p8q
7cc4d7a8dd7b2752c6b5b5a7d71628e78efa25ed: Bug 1227367 - Make harness tests compatible with mach python-test; r=automatedtester; r?gps draft
Maja Frydrychowicz <mjzffr@gmail.com> - Mon, 07 Mar 2016 16:14:40 -0500 - rev 337457
Push 12368 by mjzffr@gmail.com at Mon, 07 Mar 2016 21:20:07 +0000
Bug 1227367 - Make harness tests compatible with mach python-test; r=automatedtester; r?gps This uses the same approach as with mozunit to select pytest as the test runner. I also create a pytest plugin that modifies the pytest output to include a string expected by the ./mach python-test command. I wrote it as a named plugin, rather than just including it in conftest.py, so that it can be imported into other test suites in the future. MozReview-Commit-ID: HZWSpyjHKSr
1ca38261fc069e5963bb6a8cb7d56f4d43cc74fe: Bug 1227367 - Test exit codes in marionette-client runtests.cli; r=automatedtester draft
Maja Frydrychowicz <mjzffr@gmail.com> - Thu, 03 Mar 2016 18:10:08 -0500 - rev 337456
Push 12368 by mjzffr@gmail.com at Mon, 07 Mar 2016 21:20:07 +0000
Bug 1227367 - Test exit codes in marionette-client runtests.cli; r=automatedtester Initial tests for the Marionette Python runner/harness. MozReview-Commit-ID: 9KSSOOmpzUF
28b1b3ffb7a24e50bb6d8ce6fd6cdeefeb81cce8: Bug 1254261 - Restore icons by embedding fill colors in SVG. r=gl,bgrins draft
J. Ryan Stinnett <jryans@gmail.com> - Mon, 07 Mar 2016 14:52:02 -0600 - rev 337455
Push 12367 by bmo:jryans@gmail.com at Mon, 07 Mar 2016 20:52:32 +0000
Bug 1254261 - Restore icons by embedding fill colors in SVG. r=gl,bgrins MozReview-Commit-ID: D4JTWopI8EE
9b629d7a30f90e2a17f33f270aac12f1fc4ce459: Bug 1252303 - Creating an alarm should clear any existing alarms with the same name, r?kmag draft
bsilverberg <bsilverberg@mozilla.com> - Mon, 07 Mar 2016 15:39:30 -0500 - rev 337454
Push 12366 by bmo:bob.silverberg@gmail.com at Mon, 07 Mar 2016 20:40:32 +0000
Bug 1252303 - Creating an alarm should clear any existing alarms with the same name, r?kmag I chose to test two conditions for this bug: 1. Check that getAll() returns the expected number of alarms (i.e., creating an alarm with the same name as an existing alarm does not create a new alarm). 2. Check that the original alarm, which is replaced with an alarm with a duplicate name, is in fact cleared and therefore does not fire. This also now includes: - change alarmsMap to use Maps instead of Sets - change get() to not set lastError when an alarm does not exist - rewrite tests to expect Promises from the alarms API MozReview-Commit-ID: IWDvtbdLfuN
0330d2df344b63e05635025f8d64699867dc4f3b: Bug 1252303 - Creating an alarm should clear any existing alarms with the same name, r?kmag draft
bsilverberg <bsilverberg@mozilla.com> - Mon, 07 Mar 2016 15:29:57 -0500 - rev 337453
Push 12365 by bmo:bob.silverberg@gmail.com at Mon, 07 Mar 2016 20:33:08 +0000
Bug 1252303 - Creating an alarm should clear any existing alarms with the same name, r?kmag I chose to test two conditions for this bug: 1. Check that getAll() returns the expected number of alarms (i.e., creating an alarm with the same name as an existing alarm does not create a new alarm). 2. Check that the original alarm, which is replaced with an alarm with a duplicate name, is in fact cleared and therefore does not fire. This also now includes: - change alarmsMap to use Maps instead of Sets - change get() to not set lastError when an alarm does not exist - rewrite tests to expect Promises from the alarms API MozReview-Commit-ID: 7zAO7hc8ML9
f73a9a4f68acfe363f58f0c75f56391e4748357c: bug 1248020 - convert a bunch of moz.build files to use USE_YASM. r?mshal draft
Ted Mielczarek <ted@mielczarek.org> - Thu, 03 Mar 2016 11:38:59 -0500 - rev 337452
Push 12364 by tmielczarek@mozilla.com at Mon, 07 Mar 2016 20:29:08 +0000
bug 1248020 - convert a bunch of moz.build files to use USE_YASM. r?mshal MozReview-Commit-ID: n59TejhJq6
6eb9f29a9c56c4cc388f9f89a01d9fbc5e8ef694: bug 1248020 - add USE_YASM to moz.build. r?mshal draft
Ted Mielczarek <ted@mielczarek.org> - Tue, 01 Mar 2016 21:12:36 -0500 - rev 337451
Push 12364 by tmielczarek@mozilla.com at Mon, 07 Mar 2016 20:29:08 +0000
bug 1248020 - add USE_YASM to moz.build. r?mshal MozReview-Commit-ID: Liw4RGOXiVf
2bcd7cb6ce1604b6067a37af7b7072938e0b519e: bug 1248016 - make ASFILES rules work with .s/.asm, get rid of ASM_SUFFIX. r=mshal draft
Ted Mielczarek <ted@mielczarek.org> - Tue, 01 Mar 2016 13:47:19 -0500 - rev 337450
Push 12364 by tmielczarek@mozilla.com at Mon, 07 Mar 2016 20:29:08 +0000
bug 1248016 - make ASFILES rules work with .s/.asm, get rid of ASM_SUFFIX. r=mshal MozReview-Commit-ID: 1MK0Kud51vR
595d182023567c01fc14b7fb25fc65406296466d: Bug 1253881 - memory profiler: toggle diffing when clearing snapshots;r=fitzgen draft
Julian Descottes <jdescottes@mozilla.com> - Mon, 07 Mar 2016 21:23:32 +0100 - rev 337449
Push 12363 by jdescottes@mozilla.com at Mon, 07 Mar 2016 20:28:37 +0000
Bug 1253881 - memory profiler: toggle diffing when clearing snapshots;r=fitzgen When clearing all snapshots, if currently displaying a diff, dispatch the diffing.toggleDiffing action. MozReview-Commit-ID: 1DyPZ7tHC1X
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip