edc52ff233c42294140b0e22381f7f37cd30c345: Bug 1405359 - Add some logging code in ScrollingLayersHelper. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 18:47:24 -0400 - rev 388058
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1405359 - Add some logging code in ScrollingLayersHelper. r=jrmuizel MozReview-Commit-ID: Lb4UeUxO5HL
d3d823602aa14b79688f969fca7f1b5514b479b8: Bug 1405359 - Avoid pushing and popping identical clip stacks for adjacent display items. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 18:47:17 -0400 - rev 388057
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1405359 - Avoid pushing and popping identical clip stacks for adjacent display items. r=jrmuizel Instead of unconditionally pushing and popping clips per display item, this patch changes things so that for each recursive display list, we create an ItemClips struct. We push this onto the stack when we enter the display list, and pop it off at the end. For each display item, we check to see if the clips would actually change compared to the previous display item, and only do the pop/repush in that case. MozReview-Commit-ID: J0MCc2V9hWT
2cbf78901a3d029fc683e1ec6d00065820a902b7: Bug 1405359 - Make ScrollingLayersHelper a more stateful class. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 18:46:44 -0400 - rev 388056
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1405359 - Make ScrollingLayersHelper a more stateful class. r=jrmuizel This makes ScrollingLayersHelper a non-RAII type class, and instead adds methods to notify it of when we start processing a new transaction or a new display item within the transaction. This patch has no functional changes, it's non-obvious refactoring. MozReview-Commit-ID: GEZzCGbVqB1
1921d545ea3933100baae89600c18b2209f8a434: Bug 1405359 - Replace the mPushed* variables with a more encapsulated struct. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 18:45:28 -0400 - rev 388055
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1405359 - Replace the mPushed* variables with a more encapsulated struct. r=jrmuizel Storing the per-item clip state in a struct like this will allow us to easily compare the desired clip state across items, so we can avoid doing unnecessary work when going from one item to the next. This patch has no functional changes, it's just refactoring. MozReview-Commit-ID: GX2FX4YDusO
1345767fb2d502a9ab705ca73ccecbc7f5209c99: Bug 1405359 - Stop passing around the clip id cache in all the functions. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 18:45:04 -0400 - rev 388054
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1405359 - Stop passing around the clip id cache in all the functions. r=jrmuizel Instead just keep a ref to it as a member variable. No functional change. MozReview-Commit-ID: 9jSBdZRIGuV
aaa39a5309985d35debb0aaeaa3316dc074aa608: Bug 1410107 - Grow audioipc server slabs as required. r=kinetik
Dan Glastonbury <dan.glastonbury@gmail.com> - Wed, 25 Oct 2017 09:14:45 +1000 - rev 388053
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1410107 - Grow audioipc server slabs as required. r=kinetik MozReview-Commit-ID: zFfex3LX7K
3170d21cbd25d32466502d12119f6d9824c84bbf: servo: Merge #19011 - Use actual size for old allocation in ft_realloc (from mbrubeck:realloc); r=glennw
Matt Brubeck <mbrubeck@limpet.net> - Tue, 24 Oct 2017 18:40:20 -0500 - rev 388052
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
servo: Merge #19011 - Use actual size for old allocation in ft_realloc (from mbrubeck:realloc); r=glennw Prevents crashes from improperly freed memory. Fixes #19008, fixes #18950, fixes #18949. - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #19008. - [x] There are tests for these changes Source-Repo: https://github.com/servo/servo Source-Revision: e1dac69a4054f208accd18aa443cae19ec7eaf53
680d798e440b9492e0409c01a40d85f6022fe09a: Bug 849593 - Skip samples of active SMIL timed elements when the sample time precedes the current interval; r=dholbert
Brian Birtles <birtles@gmail.com> - Tue, 24 Oct 2017 13:06:04 +0900 - rev 388051
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 849593 - Skip samples of active SMIL timed elements when the sample time precedes the current interval; r=dholbert In some circumstances it is possible to sample a timed element in the active state with a time that precedes is current interval. One possible sequence of steps leading to this situation is as follows: 1. A timed element (e.g. <set>, <animate>) with a non-zero begin time is the child of <svg> element A (its "time container") but has yet to be sampled. 2. In order to resolve its initial interval, the timed element registers a startup milestone with its time container at time 0. 3. However, before the sample is performed where the timed element's initial current interval is resolved, <svg> element A is detached from the document tree. 4. The timed element is then attached to a different <svg> element B that has a current time greater than the begin time of the timed element and less than that of <svg> element A. 5. Since the timed element is still in its startup state it registers its startup milestone again, this time with its new time container, i.e. <svg> element B. 6. A tick occurs or the document has its style flushed such that a sample is performed. This includes running the milestone sample which causes the timed element to resolve its initial current interval. Furthermore the subsequent regular sample of the timed element causes it to transition into its active state because the current time of <svg> element B is greater than the begin time of the timed element. 7. <svg> element A is re-attached to the document. 8. When we go to run the next sample, we iterate through all time containers associated with the document's animation controller which includes both <svg> element A, and <svg> element B. 9. <svg> element A renders up its 0 milestone from step (2) since it has yet to run it. It converts this to parent time, i.e. the time space of the animation controller, which will be zero or less depending on the current time of <svg> element A when it was re-attached. 10. Since the milestone from <svg> element A will be the earliest milestone time, it will be used as the next milestone sample time. 11. The timed element is then sampled using this time, but first it is converted to a time in the time space of the timed element's time container, which is now <svg> element B. As a result of this conversion, the sample time may end up being *before* the beginning of the timed element's current interval. Since timed elements never expect the time to go backwards an assertion fails when it detects that it is active, but is being sampled before its current interval. For this particular case, ignoring the "early" sample seems to be the most appropriate action. More generally, however, we can anticipate other cases similar to this where milestones are registered that cause the sample time to temporarily go backwards. A quick audit of nsSMILTimedElement::DoSampleAt suggests that, with the code changes from this patch, that is probably ok. As an alternative we could, perhaps, try to drop and re-create all milestones when time containers are re-attached to the document tree but that would add more complexity and would not necessarily cover other similar cases of this situation. I have verified that the crashtest included in this changeset fails without the code changes also in this changeset. MozReview-Commit-ID: KKGYRayNkpo
d22b8c27e4ccfcad474213c6d9789a29aa52ca4b: Bug 1408811 followup - Disable empty-1.xul for stylo windows.
Xidorn Quan <me@upsuper.org> - Wed, 25 Oct 2017 11:01:50 +1100 - rev 388050
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1408811 followup - Disable empty-1.xul for stylo windows. MozReview-Commit-ID: Cx6rhArQm2j
718c237332ba9f7372f91ea233915ffcb3e7e486: servo: Merge #18981 - Merge request type and destination (from KiChjang:fold-type-destination); r=avadacatavra
Keith Yeung <kungfukeith11@gmail.com> - Tue, 24 Oct 2017 17:02:18 -0500 - rev 388049
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
servo: Merge #18981 - Merge request type and destination (from KiChjang:fold-type-destination); r=avadacatavra Fixes #18278. Source-Repo: https://github.com/servo/servo Source-Revision: dcd7d2fa32aa53628a9be63a2d0ea64796cd968e
841b4cc3ecce14ffe384a7bc0fa987fd5c5df3d5: Bug 1410985 - Run the pocket-library animation if the item was added, regardless of the position of the Pocket button. r=adw
Jared Wein <jwein@mozilla.com> - Tue, 24 Oct 2017 17:40:59 -0400 - rev 388048
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1410985 - Run the pocket-library animation if the item was added, regardless of the position of the Pocket button. r=adw MozReview-Commit-ID: H5jFPHeusJL
92b3336f6941f4f75ca1b92366af814596d1ac1c: Bug 1411142 Cast enums to int to silence warning about comparing enums of different types r=jld
Tom Ritter <tom@mozilla.com> - Mon, 23 Oct 2017 23:54:21 -0500 - rev 388047
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1411142 Cast enums to int to silence warning about comparing enums of different types r=jld MozReview-Commit-ID: LIbuzPuZ2mr
45f8fd6fe21a2b50d85688c5d5a1acade025ad47: Bug 1411429 - Remove unused mozjemalloc huge stats. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Wed, 25 Oct 2017 07:20:37 +0900 - rev 388046
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1411429 - Remove unused mozjemalloc huge stats. r=njn Bug 1378258 removed malloc_print_stats and bug 1379890 further removed the subsequently unused arena stats. It turns out there are also some huge stats that have been unused since bug 1378258, and that are still there, so remove them.
2fa7cfb4ce84c736d49579c385ecaf7ae44f8773: Bug 1406542 Enable PTHREAD for vpx and aom under MinGW because we do in fact have them. r=rillian
Tom Ritter <tom@mozilla.com> - Tue, 24 Oct 2017 16:57:51 -0500 - rev 388045
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1406542 Enable PTHREAD for vpx and aom under MinGW because we do in fact have them. r=rillian We were getting warnings that HAVE_PTHREAD_H was being redeclared with the wrong value. Let's correct that. MozReview-Commit-ID: KVAsYofozIT
dcf8c97d8514f6a7698124b3a0126fc0b45031c7: Bug 1411371 - Update builders to Microsoft Visual Studio 2017 15.4.1. r=dmajor,mshal
Ralph Giles <giles@mozilla.com> - Tue, 24 Oct 2017 11:39:03 -0700 - rev 388044
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1411371 - Update builders to Microsoft Visual Studio 2017 15.4.1. r=dmajor,mshal Repack of the new Visual Studio release using the packaging scripts from bug 1407678. This version also includes the pgo runtime, resolving a performance regression from the previous package. MozReview-Commit-ID: LhoVyG4IwmP
617173b7edbd5c803deee852c1bb6bfdd9373e3f: Bug 1411073 - av1: Log the output of generate_sources_mozbuild.sh. r=mshal
Ralph Giles <giles@mozilla.com> - Mon, 23 Oct 2017 14:15:11 -0700 - rev 388043
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1411073 - av1: Log the output of generate_sources_mozbuild.sh. r=mshal By default, run_process() eats the output of the child process, which means when it fails we get an exception about the return code, but none of the error messages the script printed, with makes the cause of failure opaque. Resolve this by passing a log_name parameter, which will cause the default line_handler to pass all lines of output to the local logging instance, by default at the INFO level. MozReview-Commit-ID: FIVRggeKT4f
776a7eb4c87aee4482e72eea00047de58dc68228: Bug 1411073 - av1: Clean up duplicate filenames check. r=kinetik,mshal
Ralph Giles <giles@mozilla.com> - Tue, 17 Oct 2017 10:21:58 -0700 - rev 388042
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1411073 - av1: Clean up duplicate filenames check. r=kinetik,mshal Check for filenames which differ only by .asm vs .c filename extensions when importing a new version of the libaom reference implementation of the av1 video codec. These confuse our build system as well. Also remove the obsolete vp8 and vp9 search directories, which generate warnings from 'find'. MozReview-Commit-ID: DRZL7GUrsYh
8310df49481060d94dbcdec752bcedfae1b7e099: Bug 1410655: extend WebRTC ICE Telemetry probes to 62. r=francois,mjf
Nils Ohlmeier [:drno] <drno@ohlmeier.org> - Sat, 21 Oct 2017 12:29:07 -0700 - rev 388041
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1410655: extend WebRTC ICE Telemetry probes to 62. r=francois,mjf MozReview-Commit-ID: DUnk7C0FPQA
b6cbcef3f3e1c448d27b1339cb5a42a4e077dd57: Bug 1410654: extend WebRTC Scalar Telemetry probes to 62. r=francois,mjf
Nils Ohlmeier [:drno] <drno@ohlmeier.org> - Sat, 21 Oct 2017 12:08:40 -0700 - rev 388040
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Bug 1410654: extend WebRTC Scalar Telemetry probes to 62. r=francois,mjf MozReview-Commit-ID: JbXfJdIHN1a
9c1bf5ec6b26d19db70ff38fa892c11097b9d5aa: Backed out changeset ba4a39241953 (bug 1386018) for xpcshell failures, e.g. toolkit/mozapps/extensions/test/xpcshell/test_signed_install.js on Linux debug. r=backout on a CLOSED TREE
Sebastian Hengst <archaeopteryx@coole-files.de> - Wed, 25 Oct 2017 00:32:35 +0200 - rev 388039
Push 32739 by acraciun@mozilla.com at Wed, 25 Oct 2017 09:29:21 +0000
Backed out changeset ba4a39241953 (bug 1386018) for xpcshell failures, e.g. toolkit/mozapps/extensions/test/xpcshell/test_signed_install.js on Linux debug. r=backout on a CLOSED TREE
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip