429c96e4de3234447b0aaf00214422ad84e98b83: Bug 632954 - Enable tests on Android PGO; r=jmaher
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:53:38 +0000 - rev 465145
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Enable tests on Android PGO; r=jmaher Differential Revision: https://phabricator.services.mozilla.com/D22827
de8beacc5eb45a75087c1e9b8b69276d11888d53: Bug 632954 - Add final Android PGO task; r=tomprince
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:53:31 +0000 - rev 465144
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Add final Android PGO task; r=tomprince Differential Revision: https://phabricator.services.mozilla.com/D22826
c151ebf303cad175e24bcc0965c800a9d12ecb3b: Bug 632954 - Add Android profile generation task; r=tomprince,gbrown
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:53:29 +0000 - rev 465143
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Add Android profile generation task; r=tomprince,gbrown This introduces a mozharness script, android_emulator_pgo.py, to run the profileserver suite with the PGO-instrumented Android build, and collect the profile data and jarlog. The mozharness script contains some redundancy with build/pgo/profileserver.py, but the additional requirements for Android to use adb and existing mozharness classes to control the emulator made it difficult to share the desktop profileserver implementation. Differential Revision: https://phabricator.services.mozilla.com/D22825
b96dd954a456d8088a3ceda66f51d4106f516b4a: Bug 632954 - Add Android PGO-instrumented build task; r=tomprince
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:53:19 +0000 - rev 465142
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Add Android PGO-instrumented build task; r=tomprince This is the first stage of the Android PGO task pipeline to generate an instrumented build. Differential Revision: https://phabricator.services.mozilla.com/D22824
26031d3623333bbdedd1fb388f0867bc9f7cc23c: Bug 632954 - Ignore dashes in adb error messages; r=bc
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:53:15 +0000 - rev 465141
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Ignore dashes in adb error messages; r=bc The regex should match '-' characters as well as \w to properly trim the error message if the device string contains a dash. Differential Revision: https://phabricator.services.mozilla.com/D22823
097f141a499d151e167c85dcb57e66aade7c28cb: Bug 632954 - Add support for MOZHARNESS_OPTIONS to test-linux.sh; r=tomprince
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:53:06 +0000 - rev 465140
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Add support for MOZHARNESS_OPTIONS to test-linux.sh; r=tomprince The mozharness.py transform passes in "options" parameters through the MOZHARNESS_OPTIONS environment variable. This will allow the Android PGO run task to pass in the mozharness script name to test-linux.sh Differential Revision: https://phabricator.services.mozilla.com/D22822
6f5fc0d644dd1eb83294ce41b2b47be44c2d9783: Bug 632954 - Add execute bit to test-linux.sh; r=tomprince
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:53:01 +0000 - rev 465139
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Add execute bit to test-linux.sh; r=tomprince In order to call test-linux.sh with the job-script parameter, it needs to have executable permissions. Differential Revision: https://phabricator.services.mozilla.com/D22821
53d3443e55d95af494d6c8bdc3d2d7a52c5eff1e: Bug 632954 - Explicitly set NEED_XVFB to false if need-xvfb isn't set; r=tomprince
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:52:52 +0000 - rev 465138
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Explicitly set NEED_XVFB to false if need-xvfb isn't set; r=tomprince test-linux.sh defaults to true for NEED_XVFB, while build-linux.sh defaults to false. If we are using test-linux.sh from mozharness (rather than mozharness-test), we need to explicitly set NEED_XVFB to false in order to not use xvfb. Differential Revision: https://phabricator.services.mozilla.com/D22820
503bcac735831eea887393d4759358220bcb6d99: Bug 632954 - Add Android PGO mozconfigs; r=nalexander
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:52:43 +0000 - rev 465137
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Add Android PGO mozconfigs; r=nalexander Differential Revision: https://phabricator.services.mozilla.com/D22819
142ae187478da8a6c3899ef6409cafc6e60a9115: Bug 632954 - Allow passing in env to Fennec Marionette; r=gbrown
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:52:41 +0000 - rev 465136
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Allow passing in env to Fennec Marionette; r=gbrown The Fennec process needs a few extra environment variables for PGO, notably LLVM_PROFILE_FILE and MOZ_JAR_LOG_FILE to give the locations for the profile run outputs. FennecInstance needs to pass the "env" parameter on down so it can be used by DeviceRunner. Differential Revision: https://phabricator.services.mozilla.com/D22818
0615c775a0cf6e8f98e1c051cd574c0d602a738a: Bug 632954 - Add an explicit __llvm_profile_dump() call for Android; r=snorp
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:52:31 +0000 - rev 465135
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Add an explicit __llvm_profile_dump() call for Android; r=snorp When Android shuts down the ndk process, it doesn't call the registered atexit() handlers, which is normally where the profile data gets written to file. Since the PGO test suite closes the browser when it is finished, the nativeRun routine can manually call out to __llvm_profile_dump() before returning. This method has a downside that only the profile data from the calling library gets written out, rather than for the whole process. Since we are most interested in optimizing libxul, a new hook is added in Bootstrap to make sure we get the profile data for the right library. Differential Revision: https://phabricator.services.mozilla.com/D22817
3dfc0e4f8e7c8cd48b911033222fb3c3edb9762b: Bug 632954 - Add -Wno-error=backend-plugin to support Android PGO; r=glandium
Mike Shal <mshal@mozilla.com> - Mon, 18 Mar 2019 23:52:22 +0000 - rev 465134
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 632954 - Add -Wno-error=backend-plugin to support Android PGO; r=glandium Without this flag, Android PGO profile-use builds may fail with "Function control flow change detected" errors. Differential Revision: https://phabricator.services.mozilla.com/D22816
1017d1e09b8864cc8ea41f5f29714b5a54d77f77: Backed out changeset 1b8c42577e24 (bug 1535166) for build bustages on a CLOSED TREE
Andreea Pavel <apavel@mozilla.com> - Wed, 20 Mar 2019 00:48:11 +0200 - rev 465133
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Backed out changeset 1b8c42577e24 (bug 1535166) for build bustages on a CLOSED TREE
7947b90f15808c0de5673831950fe2886558d23c: Bug 1524429 - Move build/autoconf/arch.m4 to python configure. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Tue, 19 Mar 2019 22:32:58 +0000 - rev 465132
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 1524429 - Move build/autoconf/arch.m4 to python configure. r=nalexander The file is essentially limited to ARM flag tweaks, so we make the corresponding python configure code ARM-only. In large parts, the python code sets things up similarly, although in a few places we try to be smarter about than the current autoconf-based code. The resulting config.status have been verified to only have insignificant differences on both arm and aarch64 android builds. Depends on D24000 Differential Revision: https://phabricator.services.mozilla.com/D24001
1a5a5aef01d5b745fa7ebd0185556b01f16b7c4f: Bug 1524429 - MOZ_ANDROID_CPU_ARCH doesn't need MOZ_ARCH. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Tue, 19 Mar 2019 17:09:41 +0000 - rev 465131
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 1524429 - MOZ_ANDROID_CPU_ARCH doesn't need MOZ_ARCH. r=nalexander It hasn't needed it since bug 1397776. Differential Revision: https://phabricator.services.mozilla.com/D24000
13e9a230eb255fb95dd9e0d40db978717afa6a05: Bug 1246594 - Enable ESLint rule no-throw-literal by default. r=Standard8
Ian Moody <moz-ian@perix.co.uk> - Tue, 19 Mar 2019 22:02:42 +0000 - rev 465130
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 1246594 - Enable ESLint rule no-throw-literal by default. r=Standard8 Differential Revision: https://phabricator.services.mozilla.com/D24088
458563d6a69e9874abd2fafb8889da378a59678b: Bug 1535507 - Assume that we have an empty display list building override rect for frames that support it, even if an explicit one isn't present. r=miko
Matt Woodrow <mwoodrow@mozilla.com> - Tue, 19 Mar 2019 22:24:36 +0000 - rev 465129
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 1535507 - Assume that we have an empty display list building override rect for frames that support it, even if an explicit one isn't present. r=miko If the frame supports it (stacking context + containing block for fixed), and a descendant was modified, we would have created an override dirty region with just the area of that descendant. In the case where no descendants are modified, we should use an empty rect, rather than the area inherited from our parent. This fixes the case where we forcibly build position:fixed frames (since they might async scroll differently to the rest of the page), but we only need to build the container item, not the whole frame subtree within it. Added a test that shows us building the non-intersecting position:fixed, but not items within it. Differential Revision: https://phabricator.services.mozilla.com/D23610
48726c586c4ffbf25505d117fea5c56172a6e8ce: Bug 1535507 - Don't create new display items for InvalidateLayer, since only the Layer tree should need to change. r=miko
Matt Woodrow <mwoodrow@mozilla.com> - Tue, 19 Mar 2019 22:24:20 +0000 - rev 465128
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 1535507 - Don't create new display items for InvalidateLayer, since only the Layer tree should need to change. r=miko Differential Revision: https://phabricator.services.mozilla.com/D23609
74d9f16af0bf752632cba2b70d12fcf00b8cbab0: Bug 1535507 - Don't force display item rebuilding for ActiveLayerTracker expiry, since we should only need to update the Layer tree. r=miko
Matt Woodrow <mwoodrow@mozilla.com> - Tue, 19 Mar 2019 22:24:02 +0000 - rev 465127
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 1535507 - Don't force display item rebuilding for ActiveLayerTracker expiry, since we should only need to update the Layer tree. r=miko Differential Revision: https://phabricator.services.mozilla.com/D23608
fbdd97d8e41dd07bdcef93eae73a9c7e5fd72713: Bug 1487857 - Part I: Rearranging devices setting view to new design. r=gl,flod
Micah Tigley <mtigley@mozilla.com> - Tue, 19 Mar 2019 18:59:24 +0000 - rev 465126
Push 35732 by opoprus@mozilla.com at Wed, 20 Mar 2019 10:52:37 +0000
Bug 1487857 - Part I: Rearranging devices setting view to new design. r=gl,flod This is part 1 of implementing the redesigned device settings panel. In this patch we are rearranging the existing device settings view to match the new design. Differential Revision: https://phabricator.services.mozilla.com/D15734
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 tip