aef6ff7f100d7b6748020d3c83f1303cf084e0a6: Bug 1502036 - Subtest results for raptor speedometer have incorrect lower_is_better setting; r=jmaher
Rob Wood <rwood@mozilla.com> - Fri, 16 Nov 2018 17:25:32 +0000 - rev 503261
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1502036 - Subtest results for raptor speedometer have incorrect lower_is_better setting; r=jmaher Differential Revision: https://phabricator.services.mozilla.com/D12142
6875c54fd481eeb715a96474b5babb2734423d04: Bug 1497898 - Update jstests.py wpt integration for manifest optimisations, r=bbouvier,Ms2ger
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 17:59:03 +0000 - rev 503260
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Update jstests.py wpt integration for manifest optimisations, r=bbouvier,Ms2ger Depends on D10743 Differential Revision: https://phabricator.services.mozilla.com/D11668
3dedddeb0c8d6847d9c7cb7074d7d06186182663: Bug 1497898 - Handle WindowsError trying to determine if git exists, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:43 +0000 - rev 503259
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Handle WindowsError trying to determine if git exists, r=ato Depends on D8232 Differential Revision: https://phabricator.services.mozilla.com/D10743
1a0c1f962ae46cb9669622d0ed5321398bb0de99: Bug 1497898 - Update gecko wpt manifest update to use caches, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:50 +0000 - rev 503258
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Update gecko wpt manifest update to use caches, r=ato Depends on D8231 Differential Revision: https://phabricator.services.mozilla.com/D8232
750b40bcdad7d60d10ea78f386493d962af89de1: Bug 1497898 - Update manifest before tests, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:46:49 +0000 - rev 503257
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Update manifest before tests, r=ato Depends on D8230 Differential Revision: https://phabricator.services.mozilla.com/D8231
7f54376353321e398e7389a1aa3f1165d775faf9: Bug 1497898 - Pass the manifest file directly into wpttest.from_manifest, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:46:57 +0000 - rev 503256
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Pass the manifest file directly into wpttest.from_manifest, r=ato Depends on D8229 Differential Revision: https://phabricator.services.mozilla.com/D8230
daf29e8abb1eb0f5685ed9d228afeb6b4a0c4327: Bug 1497898 - Fix the .gitignore rules, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:24 +0000 - rev 503255
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Fix the .gitignore rules, r=ato Depends on D8228 Differential Revision: https://phabricator.services.mozilla.com/D8229
fe2d962a8ed2e722266a417df17f7085aeabad3b: Bug 1497898 - Fix the cache lifecycle, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:26 +0000 - rev 503254
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Fix the cache lifecycle, r=ato The caches weren't being invalidated when the manifest itself changed. To fix this the manifest itself has to be written before the cache and the cache has to include data about the manifest that it's associated with (the mtime and path are used for this purpose). To make all this work requires a single method that can load the manifest, update it, write the manifest and write the caches. Therefore we introduce a single load_and_update method that is intended to replace all previous use of the load() or update() methods (and as a bonus handles manifest version mismatches in a single place). Depends on D8227 Differential Revision: https://phabricator.services.mozilla.com/D8228
10e90d3295ee7a29b094b42912cdb584dfd47e98: Bug 1497898 - Add a custom implementation of os.walk, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:28 +0000 - rev 503253
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Add a custom implementation of os.walk, r=ato Compared to the normal os.walk this has a couple of differences: * It returns lists of (name, stat) for filenames and directories, allowing callers to reuse the stat data without going back to the system to re-request it. * Directories are always returned as paths relative to the root, and the root itself is returned as the empty string. * It is non-recursive. There are also a few features missing that aren't required for our use cases. Depends on D8226 Differential Revision: https://phabricator.services.mozilla.com/D8227
9d61715037440d58adf237622245bb847b9a898c: Bug 1497898 - Add manifest caches for the mtime and gitignore rules, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:30 +0000 - rev 503252
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Add manifest caches for the mtime and gitignore rules, r=ato When processing the manifest using the worktree, instead of reading all files to see if the content changed, instead only process files where the mtime has been updated since the previous run. Also cache the result of running gitignore, so we can save a couple of seconds processing the gitignore rules. Depends on D8225 Differential Revision: https://phabricator.services.mozilla.com/D8226
166466f3a5fe137e02a710012babe8a14756afee: Bug 1497898 - Update the lint to the new gitignore API, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:32 +0000 - rev 503251
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Update the lint to the new gitignore API, r=ato Depends on D8224 Differential Revision: https://phabricator.services.mozilla.com/D8225
215f0ae6129888a0865d151835a86aee34368156: Bug 1497898 - Update the gitignore implementation to work as an iterator filter, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:34 +0000 - rev 503250
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Update the gitignore implementation to work as an iterator filter, r=ato This updates the gitignore implemenation to take input like os.walk but with additional stat data for the files. It also makes several useful optimistaions: * Avoid using regex when just matching a literal * Identify patterns that can only match the final component of a path and run those against that component rather than the full path. * Add the possibility of providing a dictionary of paths to gitignore statuses as a cache. This dramatically reduces the amount of time we spend in gitignore processing when updating the manifest. Depends on D8223 Differential Revision: https://phabricator.services.mozilla.com/D8224
46c50835b9972454afc33df9df569afd657430ca: Bug 1497898 - Allow the gitignore filter to work on name components only, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:36 +0000 - rev 503249
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Allow the gitignore filter to work on name components only, r=ato We end up with a lot of rules like (?:.*)/.*\.ext which are basically trying to find the last component in a path and match against that. These are rather slow to run so the easiest thing tdo is just pass in the last component of the path when we know that's the only thing the rule can match. The changes to surrounding code to use this API will be made in future commits. Depends on D8222 Differential Revision: https://phabricator.services.mozilla.com/D8223
284749dd9ee9d5e693cd86887934092a8889e1fc: Bug 1497898 - Update the .gitignore file, r=ato
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 18:48:38 +0000 - rev 503248
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Update the .gitignore file, r=ato Depends on D8221 Differential Revision: https://phabricator.services.mozilla.com/D8222
aa24577bdeae449b6b51f1184cc6aec809fc3659: Bug 1497898 - Use testfile mtimes to pre-filter files iterated over for the manifest update, r=ato
Ahilya Sinha <ahilyasinha99@gmail.com> - Fri, 16 Nov 2018 18:48:40 +0000 - rev 503247
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497898 - Use testfile mtimes to pre-filter files iterated over for the manifest update, r=ato Depends on D11667 Differential Revision: https://phabricator.services.mozilla.com/D8221
32aa5695f2ca06124214d447fda352709ea2736c: Bug 1505474 - Use downloaded manifest for wpt in jsshell, r=bbouvier,Ms2ger
James Graham <james@hoppipolla.co.uk> - Fri, 16 Nov 2018 15:23:21 +0000 - rev 503246
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1505474 - Use downloaded manifest for wpt in jsshell, r=bbouvier,Ms2ger web-platform-tests gets its test list from a manifest file that's generated from the content of the tests. Unfortunately generating this manifest is slow, so it's unreasonable to create it from scratch for every test run. Until recently the generated manifest was kept in-tree, which was suboptimal in a few ways: * The manifest tended to get out of sync with the actual source * The large json file caused problems for tooling including source control and the review frontends. We previously switched `mach wpt` to download a manifest on demand and apply an incremental update. However this work missed the usage in jstests.py. This continued to use the increasing outdated in-tree manifest, which causes a number of problems * It doesn't have an up-to-date list of tests * It blocks removing that file * It blocks landing various optimisations to make updating the manifest faster. This patch fixes jstests.py to use a downloaded manifest. Unlike the tests run through a mach frontend jstests.py doesn't know where the objdir is, so it's hard to work out where to download the manifest. This patch adopts a heuristic approach; if the path to the jsshell looks like <root>/dist/bin and <root>/_tests exists, we assume it's a gecko-like objdir and use <root>/_tests/web-platform/ for the manifest; otherwise we just put it into the system tempdir. Because the manifest has to be updated on startup, this patch causes a startup time regression, but this will be considerably reduced by the work in Bug 1497898 for which this is a prerequisite. Differential Revision: https://phabricator.services.mozilla.com/D11667
f38aa27974eea413de93123272629af54443ec0b: bug 1502841 - follow-up to ensure reentering DataStorageSharedThread::Shutdown() doesn't cause problems r=jcj
Dana Keeler <dkeeler@mozilla.com> - Fri, 16 Nov 2018 16:14:52 +0000 - rev 503245
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
bug 1502841 - follow-up to ensure reentering DataStorageSharedThread::Shutdown() doesn't cause problems r=jcj We shouldn't reenter DataStorageSharedThread::Shutdown(), but it may be possible. To guard against potentially attempting to shut down the shared thread more than once, we can check gDataStorageSharedThreadShutDown first. Differential Revision: https://phabricator.services.mozilla.com/D12050
2c6f77ee5cf5b4f387ea870fda2a80f24d028b25: Bug 1493766 - part4 : add test. r=johannh
alwu <alwu@mozilla.com> - Wed, 14 Nov 2018 01:36:26 +0000 - rev 503244
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1493766 - part4 : add test. r=johannh Add a test to ensure the temporary autoplay permission is sync correctly between JS and C++ side. Differential Revision: https://phabricator.services.mozilla.com/D11711
7a9c6dea709ee75cef4d10beb79aeece3cf64d1f: Bug 1493766 - part3 : modify test. r=smaug
alwu <alwu@mozilla.com> - Wed, 14 Nov 2018 01:32:26 +0000 - rev 503243
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1493766 - part3 : modify test. r=smaug Differential Revision: https://phabricator.services.mozilla.com/D7017
41fe6af27564c37af484fdad90833a065e30c314: Bug 1493766 - part2 : save the status of temporary autoplay permission in outer window. r=johannh,smaug
alwu <alwu@mozilla.com> - Fri, 16 Nov 2018 18:27:00 +0000 - rev 503242
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1493766 - part2 : save the status of temporary autoplay permission in outer window. r=johannh,smaug In order to know whether we have temporary autoplay permission without creating a request, we need to cache its state in the outer window so that we can get the correct returned value for AutoplayPolicy::IsAllowedToPlay(). Differential Revision: https://phabricator.services.mozilla.com/D7013
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip