0943280cce46c64aea866510d45c809a28ee1457: servo: Merge #16845 - Renamed BrowsingContext to WindowProxy in script (from asajeffrey:script-rename-browsing-contexts); r=jdm
Alan Jeffrey <ajeffrey@mozilla.com> - Sat, 13 May 2017 07:09:29 -0500 - rev 358178
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16845 - Renamed BrowsingContext to WindowProxy in script (from asajeffrey:script-rename-browsing-contexts); r=jdm <!-- Please describe your changes on the following line: --> Renamed `script::dom::BrowsingContext` to `script::dom::WindowProxy`. The browsing context is mostly maintained in the constellation, not in script. It would be nice to rename `constellation::Frame` to `constellation::BrowsingContext`, but that will be very confusing if there are two `BrowsingContext` types. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes do not require tests because renamings aren't externally visible <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Source-Repo: https://github.com/servo/servo Source-Revision: 34d0e59849a0a3e231e47fe10d66484340b8b80c
0ab23ad751cf27789e0f9c114d9033eae28ae011: Bug 1363984 - Update mochitest expectations for this bug.
Xidorn Quan <me@upsuper.org> - Sat, 13 May 2017 22:33:52 +1000 - rev 358177
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1363984 - Update mochitest expectations for this bug. MozReview-Commit-ID: B6AShaCXwv1
1b72e0fc7e42ed51e6b47c26c809f7f1becca234: servo: Merge #16836 - Make `./mach rustup` use the latest nightly rather than master (from servo:nightly); r=nox
Simon Sapin <simon.sapin@exyr.org> - Sat, 13 May 2017 06:07:11 -0500 - rev 358176
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16836 - Make `./mach rustup` use the latest nightly rather than master (from servo:nightly); r=nox Source-Repo: https://github.com/servo/servo Source-Revision: d2fa2ae9343db8d958570a501ff9a7b7a203ceeb
ddb010a0cf6bd41573f55b6f576ea651aecc7bdc: servo: Merge #16827 - Make font-feature-settings a subprop of font (from upsuper:subprop-font-feature-settings); r=Manishearth
Xidorn Quan <me@upsuper.org> - Sat, 13 May 2017 04:40:18 -0500 - rev 358175
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16827 - Make font-feature-settings a subprop of font (from upsuper:subprop-font-feature-settings); r=Manishearth Source-Repo: https://github.com/servo/servo Source-Revision: e83e102736907a5905134aa129dd43debf5c216c
fb95c37e1aed159d897b9394800bff5e367f762d: Bug 1360500 - Allow custom colors on find selection type selections. r=jaws,masayuki,smaug
Timothy Guan-tin Chien <timdream@gmail.com> - Wed, 10 May 2017 10:48:50 -0400 - rev 358174
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1360500 - Allow custom colors on find selection type selections. r=jaws,masayuki,smaug This patch implements chrome-only Selection#setColors and Selection#resetColors methods, and use it to set the background color of the preferences search highlight. MozReview-Commit-ID: 2U92aBCAyeh
426c2e8d1d49e6abdd013d6c6880364a705c37b5: servo: Merge #16843 - explicitly specify bitness for x86 cross-compilation situations (from froydnj:bindgen-cross-compile); r=emilio
Nathan Froyd <froydnj@gmail.com> - Sat, 13 May 2017 02:03:19 -0500 - rev 358173
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16843 - explicitly specify bitness for x86 cross-compilation situations (from froydnj:bindgen-cross-compile); r=emilio Despite compiling for architecture X, the user may have specified a clang that defaults to architecture Y. We need to ensure that we invoke clang with the correct architecture selection. We do not use --target to do this, however, because that runs into problems with LLVM's default search paths. For the x86 case, we can simply use -m$BITNESS to select the correct architecture. This is not at all a general solution, but it does unblock compiling Stylo for 32-bit Linux on Gecko's infra. - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). Source-Repo: https://github.com/servo/servo Source-Revision: 9be1fead3ccf419e5d32a935ba6a5271c2e5170a
39581b9d0093df165534ccb04f926177d29d7192: Backed out 2 changesets (bug 1363421) for assertions in test_enterjit_osr.js on Windows xpcshell tests a=backout
Wes Kocher <wkocher@mozilla.com> - Sat, 13 May 2017 00:32:36 -0700 - rev 358172
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Backed out 2 changesets (bug 1363421) for assertions in test_enterjit_osr.js on Windows xpcshell tests a=backout Backed out changeset 722236d77865 (bug 1363421) Backed out changeset 8a9e10fae26c (bug 1363421) MozReview-Commit-ID: 93rH7BqbgHP
3c7136c3ad2feacbe943d19d49ab8c005f9635d4: Bug 1341102 - Update reftest expectations. r=me
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Sat, 13 May 2017 16:24:22 +0900 - rev 358171
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1341102 - Update reftest expectations. r=me https://github.com/servo/servo/issues/16824 MozReview-Commit-ID: LmuEyDj6wRi
040336fc37c9dc1fb926da0a5e067d923849ef84: servo: Merge #16841 - Fix the situation when :any-link starts or stops matching to actually restyle correctly (from bzbarsky:fix-moz-any-matching); r=emilio
Boris Zbarsky <bzbarsky@mit.edu> - Sat, 13 May 2017 01:00:46 -0500 - rev 358170
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16841 - Fix the situation when :any-link starts or stops matching to actually restyle correctly (from bzbarsky:fix-moz-any-matching); r=emilio <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix https://bugzilla.mozilla.org/show_bug.cgi?id=1364335 <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Source-Repo: https://github.com/servo/servo Source-Revision: 3b13b152a0198b9a005ca122e6cb411b12f8c334
503e83d0d7ab6c3e53cf4f5fab016ee305e76351: Bug 1361258 - Backout 2214b3c57c9c (bug 1291356), disabling multiple jemalloc arenas. r=glandium
Bobby Holley <bobbyholley@gmail.com> - Sat, 13 May 2017 08:21:05 +0200 - rev 358169
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1361258 - Backout 2214b3c57c9c (bug 1291356), disabling multiple jemalloc arenas. r=glandium
973ceec1e2c4bd4a6ae65698d66916a53e475477: servo: Merge #16824 - Recalculate font-size when generic changes (from Manishearth:font-size-generic-changed); r=heycam
Manish Goregaokar <manishsmail@gmail.com> - Fri, 12 May 2017 23:59:25 -0500 - rev 358168
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16824 - Recalculate font-size when generic changes (from Manishearth:font-size-generic-changed); r=heycam With the MathML refactorings this feature got lost. It would still get recalculated when explicitly specified as a keyword, but not otherwise. To avoid hitting the font metrics provider too often, we only do this when the generic changes. Otherwise we trust the existing calculated font. I swear, once Stylo lands I'm going to campaign to remove font-size from CSS entirely. :weary: Source-Repo: https://github.com/servo/servo Source-Revision: 29f5b226ac6029cfa3806a36e58974b94c12d655
9efa29ce0e3edc8eb84c906328e3d08236fee6da: Bug 1364338: Force a repaint when CSS 'outline-width' or 'outline-offset' change. r=heycam
Daniel Holbert <dholbert@cs.stanford.edu> - Fri, 12 May 2017 18:42:10 -0700 - rev 358167
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1364338: Force a repaint when CSS 'outline-width' or 'outline-offset' change. r=heycam Previously, when these properties changed, we'd only send change hints to recompute overflow areas & trigger DLBI. If the outline was always outside of the element's border box, this old strategy was generally OK, because the outline tweak would cause a change to the overflow areas' size, and that would invalidate the changed area via DLBI & trigger a repaint. However, for outlines that are *inside* of the element (via negative 'outline-offset'), these change hints were not sufficient, because tweaks to the outline width & offset will NOT affect the size of the element's overflow areas and will not trigger any DLBI invalidation. So in order to correctly handle these changes, we really need to request a repaint of the affected element, since some piece of the element may need to be repainted even if it's not changing in size. MozReview-Commit-ID: J4KGUHrJ09U
563f87448eff9e4db7c11f9bfbfddc4c746ca514: Bug 1316956 - Add a when argument to check_prog. r=chmanchester
Mike Hommey <mh+mozilla@glandium.org> - Sat, 13 May 2017 07:12:56 +0900 - rev 358166
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1316956 - Add a when argument to check_prog. r=chmanchester
d1ed0a4e7d97f808c1caa1ebc5116b51d1986621: servo: Merge #16837 - Fix future illegal_floating_point_literal_pattern warnings (from servo:illegal_floating_point_literal_pattern); r=emilio
Simon Sapin <simon.sapin@exyr.org> - Fri, 12 May 2017 22:34:09 -0500 - rev 358165
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16837 - Fix future illegal_floating_point_literal_pattern warnings (from servo:illegal_floating_point_literal_pattern); r=emilio They make component/style fail to build, because of `#[deny(warnings)]` Source-Repo: https://github.com/servo/servo Source-Revision: b6f5d65bbddc9e8d35016c8669256a8a539b7516
942071ef8ff7adbc9a2cc3d9d8a06aebfff941d2: servo: Merge #16833 - Fix unsafe AtomicRefCell<T> transmutes in Layout component (from MortimerGoro:layout_alignment); r=SimonSapin
Imanol Fernandez <mortimergoro@gmail.com> - Fri, 12 May 2017 20:44:02 -0500 - rev 358164
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16833 - Fix unsafe AtomicRefCell<T> transmutes in Layout component (from MortimerGoro:layout_alignment); r=SimonSapin <!-- Please describe your changes on the following line: --> Fixes unsafe transmute between `AtomicRefCell<PersistentLayoutData>` and `AtomicRefCell<PartialPersistentLayoutData>` which have different memory alignment in 32 bit archs leading to SEGV crashes. See https://github.com/servo/servo/issues/16817 and https://github.com/servo/servo/pull/16816 mem::align_of values in 32 bit archs (e.g. Android): ``` PersistentLayoutData 8 PersistentLayoutData 4 AtomicRefCell<PersistentLayoutData> 8 AtomicRefCell<PartialPersistentLayoutData> 4 ``` mem::align_of values in 64 bit archs ``` PersistentLayoutData 8 PersistentLayoutData 8 AtomicRefCell<PersistentLayoutData> 8 AtomicRefCell<PartialPersistentLayoutData> 8 ``` --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #16817 (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Source-Repo: https://github.com/servo/servo Source-Revision: 47e4c48feb12e4189f11fd94631f0abea5827f91
edea8382d3f45ed2fdd38c42f210a63c139f3be2: Bug 1360429 - Part 2. Detect __ARM_NEON macro for aarch64. r=glandium
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Fri, 28 Apr 2017 13:27:10 +0900 - rev 358163
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1360429 - Part 2. Detect __ARM_NEON macro for aarch64. r=glandium aarch64's gcc and arm's gcc with -mfpu=neon defines __ARM_NEON for NEON, so we should detect it to support NEON code. MozReview-Commit-ID: LRMTQLctuLV
44e7fc73aeb508c0a363319ec48a5db4b175bb9b: Bug 1360429 - Part 1. inline arm assember for YCBCR should be on arm only. r=sotaro
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Fri, 28 Apr 2017 13:22:07 +0900 - rev 358162
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1360429 - Part 1. inline arm assember for YCBCR should be on arm only. r=sotaro ycvcr_to_rgb565 uses inline assember for arm neon. Since it is different for aarch64's assembler, we should define HAVE_YCBCR_TO_RGB565 on arm32 only. MozReview-Commit-ID: 4c2n1luvVvC
3aab38317ccd9db6184682188504ae0f65ac20df: Bug 1364635 - Don't offer to configure Mercurial if running from a Git checkout; r=Ehsan
Gregory Szorc <gps@mozilla.com> - Fri, 12 May 2017 19:44:03 -0700 - rev 358161
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1364635 - Don't offer to configure Mercurial if running from a Git checkout; r=Ehsan Previously, `mach bootstrap` would unconditionally prompt to help configure Mercurial in most scenarios. I agree with Ehsan's observation in a mailing list post that this behavior doesn't make sense when running from a Git checkout, as the user probably doesn't care about Mercurial if they are using Git. This change doesn't completely ignore Mercurial for Git users. For example, we still unconditionally run code that verifies that Mercurial is installed and reasonably up to date. Changing this would be a bit of work. But even if we wanted to change it, git-cinnabar users would benefit from having a modern Mercurial installed. So it isn't straightforward for Git users to ignore Mercurial completely. MozReview-Commit-ID: 8ncHRgCsjz
7769ede8cd8ceaf446a882b48e0034887503ba92: servo: Merge #16847 - Opt-in to thread-local jemalloc arenas for stylo rayon threads (from glandium:master); r=bholley
Mike Hommey <mh+mozilla@glandium.org> - Fri, 12 May 2017 19:11:31 -0500 - rev 358160
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
servo: Merge #16847 - Opt-in to thread-local jemalloc arenas for stylo rayon threads (from glandium:master); r=bholley <!-- Please describe your changes on the following line: --> Servo part of https://bugzilla.mozilla.org/show_bug.cgi?id=1361258 --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [ ] `./mach test-tidy` does not report any errors <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because they're gecko-specific <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Source-Repo: https://github.com/servo/servo Source-Revision: 642cd08f21727ee35dc3dace14d0c9ad5837f380
80b6cb6d9ee73e05500980428c2dc2c2632dcab4: Bug 1362388 - Remove nsAHttpTransaction::Available as footgun r=nwgh
Patrick McManus <mcmanus@ducksong.com> - Fri, 05 May 2017 18:12:36 -0400 - rev 358159
Push 31818 by archaeopteryx@coole-files.de at Sun, 14 May 2017 16:01:21 +0000
Bug 1362388 - Remove nsAHttpTransaction::Available as footgun r=nwgh nsAHttpTransaction::Available() obtained a bytecount from the abstract transaction's input stream. If that stream was derived from a file:// it would create janky IO - so remove the interface. Http2Push maintains a non-inherited interface which is used to check the number of bytes it has internally buffered in memory. MozReview-Commit-ID: IQHt8yGsqDE
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip