300a08d9762e3efcfdedb292d540420ba9c9083f: Bug 1497388 clarify documentation of out-parameter values for wrappers and proxies r=jwalden
Karl Tomlinson <karlt+@karlt.net> - Wed, 10 Oct 2018 17:49:50 +0000 - rev 499062
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1497388 clarify documentation of out-parameter values for wrappers and proxies r=jwalden Differential Revision: https://phabricator.services.mozilla.com/D8049
70505741627fe29e79facb7fab0e3297320f0421: Bug 1496220 - 3. Disable Necko IPC security for test_saveHeapSnapshot_e10s_01; r=fitzgen
Jim Chen <nchen@mozilla.com> - Tue, 09 Oct 2018 16:30:46 +0000 - rev 499061
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1496220 - 3. Disable Necko IPC security for test_saveHeapSnapshot_e10s_01; r=fitzgen The use of <iframe mozbrowser> in test_saveHeapSnapshot_e10s_01.html has implicitly depended on the "network.disable.ipc.security" pref set to false, which is the default for desktop and not applicable for Fennec. With the new mobile test harness, this pref needs to be set explicitly by the test. Differential Revision: https://phabricator.services.mozilla.com/D7786
daf7ced3bbccbbf3a17d2bdc3d65d97def353762: Bug 1496220 - 2. Default to null triggering principal for GeckoView.loadUri; r=esawin
Jim Chen <nchen@mozilla.com> - Fri, 05 Oct 2018 13:43:05 +0000 - rev 499060
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1496220 - 2. Default to null triggering principal for GeckoView.loadUri; r=esawin For improved security, default to a null triggering principal for GeckoView.loadUri calls, except when loading certain privileged schemes such as "resource" and "file". Differential Revision: https://phabricator.services.mozilla.com/D7785
abf2e04e30e4bb0b9cb534926e4df492b9fecf0b: Bug 1496220 - 1. Serialize default null principal in RemoteWebNavigation; r=jkt
Jim Chen <nchen@mozilla.com> - Fri, 05 Oct 2018 13:40:43 +0000 - rev 499059
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1496220 - 1. Serialize default null principal in RemoteWebNavigation; r=jkt RemoteWebNavigation is expected to pass a serialized principal, so the default null principal should be serialized as well. Differential Revision: https://phabricator.services.mozilla.com/D7784
f5c1a7734493d3cf9e1ded5731bba0528b01155f: bug 1483651 - Dereference symlinks inside vendored Python packages. r=gps
Ted Mielczarek <ted@mielczarek.org> - Wed, 10 Oct 2018 19:59:52 +0000 - rev 499058
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
bug 1483651 - Dereference symlinks inside vendored Python packages. r=gps At least one Python package that got vendored via `mach vendor python` contains a symlink, which hg.mozilla.org rejects. This change makes it so symlinks get replaced with the contents of the file they point to. Differential Revision: https://phabricator.services.mozilla.com/D4135
ec133330b1ec04135b786d0d3dc9a2fe4460d874: bug 1481612 - Add more actions to virtualenv_packages.txt and use them to include the unpacked Windows psutil wheel. r=gps
Ted Mielczarek <ted@mielczarek.org> - Wed, 10 Oct 2018 19:53:47 +0000 - rev 499057
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
bug 1481612 - Add more actions to virtualenv_packages.txt and use them to include the unpacked Windows psutil wheel. r=gps This patch adds two new actions to virtualenv_packages.txt processing: windows and !windows. The former processes the rest of the action only on Windows, and the latter processes it only on non-Windows. These new features are used in virtualenv_packages.txt to use the path to the unpacked Windows psutil wheel when on Windows, and build psutil from source and use that path on other platforms. This fixes the long-standing problem of not having psutil available on most Windows systems (since they don't have the right set of Visual C++ build tools). Differential Revision: https://phabricator.services.mozilla.com/D3437
7243432584616e49d01a0b8910518b359d5aab99: bug 1481612 - vendor an unpacked win64 wheel of psutil 5.4.3. r=gps
Ted Mielczarek <ted@mielczarek.org> - Wed, 10 Oct 2018 19:54:25 +0000 - rev 499056
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
bug 1481612 - vendor an unpacked win64 wheel of psutil 5.4.3. r=gps This change uses the previously added `--with-windows-wheel` option to vendor a win64 wheel of psutil. This patch was produced by running: `mach vendor python --with-windows-wheel psutil==5.4.3`. We vendor this wheel unpacked so that we can insert it into sys.path in mach without needing a virtualenv in which to install a wheel with pip, since we have a chicken-and-egg problem there where configure creates the virtualenv but we'd like to be able to use psutil before we run configure. Differential Revision: https://phabricator.services.mozilla.com/D3436
60407ad1392236779a537033965c59e3170416f3: bug 1481612 - Add a --with-windows-wheel option to mach vendor python. r=gps
Ted Mielczarek <ted@mielczarek.org> - Wed, 10 Oct 2018 19:54:36 +0000 - rev 499055
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
bug 1481612 - Add a --with-windows-wheel option to mach vendor python. r=gps This option is very single-purpose: it's intended to let us vendor an unpacked wheel for psutil on Windows. To that end the mach command will error if you try to use it for anything but vendoring a single package. The mach command will vendor source packages as it currently does, and then run `pip download` again with some hardcoded parameters to fetch the right wheel for Python 2.7 on win64 and unpack it to a `package-platform` directory under `third_party/python`. I don't expect this to be used for anything but psutil, but it should make life simpler for anyone that wants to update our vendored copy of psutil in the future. Differential Revision: https://phabricator.services.mozilla.com/D3435
aafdbf2213ec4463e830f810dbb71016f5e49c5f: bug 1481612 - Re-vendor psutil 5.4.3 using `mach vendor python`. r=chmanchester,gps
Ted Mielczarek <ted@mielczarek.org> - Wed, 10 Oct 2018 19:47:58 +0000 - rev 499054
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
bug 1481612 - Re-vendor psutil 5.4.3 using `mach vendor python`. r=chmanchester,gps psutil was previously vendored manually, so re-vendor it using our new tooling to make for smaller diffs in follow-up patches. This mostly just winds up adding some extra files that must have been left out of the manual vendoring, but it also updates the third_party/python/requirements.{in,txt}. Differential Revision: https://phabricator.services.mozilla.com/D3434
161743a69d9a74f2cc5ea7cfc2e8f65c56a4ec2a: Bug 1492127 - Split browser_blockingIndexedDb.js into browser_blockingIndexedDb.js and browser_blockingIndexedDbInWorkers.js to give it more breathing room timeout-wise r=baku
Ehsan Akhgari <ehsan@mozilla.com> - Wed, 10 Oct 2018 19:19:09 +0000 - rev 499053
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1492127 - Split browser_blockingIndexedDb.js into browser_blockingIndexedDb.js and browser_blockingIndexedDbInWorkers.js to give it more breathing room timeout-wise r=baku Depends on D8175 Differential Revision: https://phabricator.services.mozilla.com/D8176
bea77a4e96999378c7c8dc1dbc90bbc585171ca9: Bug 1479912 - Don't rasterize blobs in parallel when there isn't a lot of blobs. r=jrmuizel
Nicolas Silva <nsilva@mozilla.com> - Wed, 10 Oct 2018 18:12:00 +0000 - rev 499052
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1479912 - Don't rasterize blobs in parallel when there isn't a lot of blobs. r=jrmuizel Differential Revision: https://phabricator.services.mozilla.com/D8263
ad50275695e8d42a36e5f3151be4ee955a5185c2: Bug 1497972 - Use LookupForAdd in ImageLoader::LoadImage. r=mats
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 10 Oct 2018 19:09:59 +0000 - rev 499051
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1497972 - Use LookupForAdd in ImageLoader::LoadImage. r=mats I think it's not really that awkward :) Differential Revision: https://phabricator.services.mozilla.com/D8268
9752f179b9c3007eca80f96585dcfd3d951ae222: Bug 1494069 - [lint] Explicitly list out objdirs rather than depend on 'obj*' in the global exclude, r=rwood
Andrew Halberstadt <ahalberstadt@mozilla.com> - Fri, 05 Oct 2018 19:48:53 +0000 - rev 499050
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1494069 - [lint] Explicitly list out objdirs rather than depend on 'obj*' in the global exclude, r=rwood When using globs in exclude directorives, FileFinder will return every *file* that gets matched. This is can be thousands of files in the case of an objdir. While we now collapse these files down to highest possible directories, this collapse operation can still take a noticeable amount of time (0.6s). This simply scans topsrcdir for files that start with 'obj' to avoid the glob. This also moves the '_activate_virtualenv' call to the top of the function because in CI, this will cause an objdir to be created (to store the virtualenv). If this happens *after* calculating the global excludes, we won't catch it since it doesn't exist yet. This will result in the objdir's virtualenv being linted and erroneous failures. Depends on D7739 Differential Revision: https://phabricator.services.mozilla.com/D7740
fe0fb280dbfc24091da4fd2627c6d514fbbab13f: Bug 1494069 - [mozlint] Collapse exclude paths into their smallest possible set, r=egao
Andrew Halberstadt <ahalberstadt@mozilla.com> - Tue, 09 Oct 2018 19:26:23 +0000 - rev 499049
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1494069 - [mozlint] Collapse exclude paths into their smallest possible set, r=egao Often we specify globs in our exclude patterns, e.g: exclude: - **/node_modules - obj* However, these globs get expanded out to *every* file that matches them. This can sometimes be thousands or even tens of thousands of files. We then pass these paths on to the underlying linters and tell them to exclude them all. This causes a lot of overhead and slows down performance. This commit implements a "collapse" function. Given a set of paths, it'll collapse them into the smallest set of parent directories that contain the original set, and that don't contain any extra files. For example, given a directory structure like this: a -- foo.txt -- b -- bar.txt -- baz.txt -- c -- ham.txt -- d -- spam.txt Then the following will happen: >>> collapse(['a/foo.txt', 'a/b/bar.txt', 'a/c/ham.txt', 'a/c/d/spam.txt']) ['a/foo.txt', 'b/bar.txt', 'c'] Since all files under directory 'c' are specified by the original set (both 'c/ham.txt' and 'c/d/spam.txt'), we can collapse it down to just 'c'. However not all files under 'b' are specified (we're missing 'a/b/baz.txt'), so we can't collapse 'b' (and by extension also can't collapse 'a'). If we had included 'a/b/baz.txt': >>> collapse(['a/foo.txt', 'a/b/bar.txt', 'a/b/baz.txt', 'a/c/ham.txt', 'a/c/d/spam.txt']) ['a'] In both cases, the smallest set of paths that contains the original set (and only the original set) is computed. The collapse function has a little bit of overhead but it's not too bad. For example collapsing all files matched by '**/node_modules' takes ~0.015s. Collapsing two full objdirs, takes ~0.6s. But a follow up commit is planned to make sure we stop using 'obj*' to reduce that overhead. Depends on D7738 Differential Revision: https://phabricator.services.mozilla.com/D7739
a2956764213e874d77c1541d71a48c3fffb5725d: Bug 1494069 - Vendor scandir==1.9.0 to third_party/python, r=davehunt
Andrew Halberstadt <ahalberstadt@mozilla.com> - Wed, 10 Oct 2018 19:21:07 +0000 - rev 499048
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1494069 - Vendor scandir==1.9.0 to third_party/python, r=davehunt Scandir is a faster implementation of os.listdir since it caches file metadata as it works. Using listdir and then calling things like os.path.isfile after the fact will result in multiple system calls. Whereas with scandir, there will only be a single system call per file. Scandir is part of the stdlib in Python 3.6+, so the following can be used for Python 2/3 compatible code: try from os import scandir, walk except ImportError: from scandir import scandir, walk Differential Revision: https://phabricator.services.mozilla.com/D7738
2084a929ec4cbf31dda4df9935ba4bb38002a6bd: Bug 1492866 - add accessible relations to accessibility panel sidebar. r=pbro
Yura Zenevich <yura.zenevich@gmail.com> - Wed, 10 Oct 2018 19:18:03 +0000 - rev 499047
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1492866 - add accessible relations to accessibility panel sidebar. r=pbro MozReview-Commit-ID: 5u8SRnlnFyN Differential Revision: https://phabricator.services.mozilla.com/D7208
da3163c05c7ee076a6f91c25d4fe5bf5bcb13429: Bug 1494401 - Split browser_blockingWorkers.js into three separate tests to give it more breathing room timeout-wise r=baku
Ehsan Akhgari <ehsan@mozilla.com> - Wed, 10 Oct 2018 19:19:07 +0000 - rev 499046
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1494401 - Split browser_blockingWorkers.js into three separate tests to give it more breathing room timeout-wise r=baku Differential Revision: https://phabricator.services.mozilla.com/D8175
81dd098adbd8984f9491eae22dac152cf4fb2d71: Bug 1492977: Mark returns-false-for-retry functions as Pure r=tcampbell
Iain Ireland <iireland@mozilla.com> - Wed, 10 Oct 2018 19:18:10 +0000 - rev 499045
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1492977: Mark returns-false-for-retry functions as Pure r=tcampbell Differential Revision: https://phabricator.services.mozilla.com/D7942
2c21c87e773ee1bfd796079fe5ffe06c2057b9ae: Bug 1497686 - Better error message when a jsapi-test fails with an exception pending. r=jwalden
Jason Orendorff <jorendorff@mozilla.com> - Wed, 10 Oct 2018 17:47:54 +0000 - rev 499044
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1497686 - Better error message when a jsapi-test fails with an exception pending. r=jwalden Differential Revision: https://phabricator.services.mozilla.com/D8154
71160e8bcfb743129388fa3d3948a8d036263785: Bug 1496362 - Limit control of tracelogger to only the gecko profiler and the previous environment variables implementation that triggers tracelogger only on startup. r=sfink
Denis Palmeiro <dpalmeiro@mozilla.com> - Wed, 10 Oct 2018 18:36:53 +0000 - rev 499043
Push 1864 by ffxbld-merge at Mon, 03 Dec 2018 15:51:40 +0000
Bug 1496362 - Limit control of tracelogger to only the gecko profiler and the previous environment variables implementation that triggers tracelogger only on startup. r=sfink Tracelogger is currently not setup to be toggled on and off while in the shell. Removing access to the "jit.enable-tracelogger" will force the tracelogger to be invoked either through the new tracelogger API which is intended to be used by the profiler, or through setting the environment variables TLDIR, TLOPTIONS, and TLLOG. Differential Revision: https://phabricator.services.mozilla.com/D8250
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip