Merge mozilla-central to autoland a=merge on a CLOSED TREE

servo: Merge #19884 - Fix mutation test in CI by passing env vars correctly (from aneeshusa:fix-mutation-test); r=jdm Originally, buildbot would try to directly execute `RUSTFLAGS=` as a binary since it was the first token in the command string, which won't work. Fix this by using the `env` command to set the environment variables, as there is currently no support for setting per-step environment variables in the `buildbot_steps.yml` file. <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [ ] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes should fix #19821 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [x] These changes do not require tests because they change the CI configuration <!-- 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: 025e5d773313e16ffe73ae63efc068b79b82d531

Bug 1433591: Test that dynamic changes properly affect hit-testing for elements that have anon boxes. r=bz MozReview-Commit-ID: 3NfL4Lf7yOu

Bug 1433591: Update the style of child anon box wrapper after owned anon boxes. r=bz Since the former could inherit from the latter. Test incoming in a followup. MozReview-Commit-ID: LJjprcyjDyU

Bug 1428692 - Don't use tweakExpectedRestyleCount in the case where we don't use the returned count for observeStyling called right after. r?birtles tweakExpectedRestyleCount() is supposed to work with observeStyle() which is called right after tweakExpectedRestyleCount(). That's because tweakExpectedRestyleCount() adjusts the expected restyle count which will happen in continuous frames that begins from the startsRightNow() call, especially a redundant restyle might be observed in the last frame if the animation did not begin at the current timeline time and if the Promise inside the last requestAnimationFrame is fulfilled after restyling happened. Instead of using tweakExpectedRestyleCount(), we need to check whether the animation begun at the current timeline time when the animation started, and if the animation begun at the current timeline time, we don't observe the superfluous restyle in a frame after the animation element was re-attached. MozReview-Commit-ID: 6TLQERSSbjU

Bug 1193394 - Part 1: Microtasks and promises scheduling. r=bevis MozReview-Commit-ID: 7nAQUJSiBg5

Bug 1415780 - Drop CollectDocuments in nsRefreshDriver.cpp. r?hiro MozReview-Commit-ID: ETSPKpWDuI4

Bug 1415780 - Call UpdateTiming() prior to remove the animtion from the timeline in Animation::CancelNoUpdate. r?birtles Now AnimationEventDispatcher ensures that the refresh driver's next tick happens for cancel event, so we don't need to re-observe the timeline (it happens in UpdateTiming) once after removing the animation from the timeline. MozReview-Commit-ID: 7ivclmYIkPa

Bug 1415780 - Let AnimationEventDispatcher observe nsRefreshDriver. r?birtles So that we can now ensure nsRefreshDriver ticks (i.e. nsRefreshDriver doesn't stop its timer) for all queued events. Before this patch, dispatching CSS animation/transition events relied on the fact that DocumentTimeline observes nsRefreshDriver. For this fact, animationcancel or transitioncancel event did not dispatch properly in some cases, i.e. the case where the animation was dropped from the DocumentTimeline. MozReview-Commit-ID: 7JYro0MY2U2

Bug 1415780 - Make AnimationEventDispatcher::SortEvents() private. r?birtles Now we sort pending events only when dispatching the events, i.e. only inside DispatchEvent(). MozReview-Commit-ID: 1TZ1JhqQBxN

Bug 1415780 - Let nsPresContext have AnimationEventDispatcher. r?birtles Now we sort CSS animation/transition events by scheduled event time prior to compositor order. SortEvents() will be a private method in the next patch in this patch series. MozReview-Commit-ID: ICkOayquN0f

Bug 1415780 - Make AnimationEventDispatcher refcountable. r?birtles In a subsequent patch in this patch series, we want to make nsPresContext have an AnimationEventDispatcher as RefPtr<>. Instead, if we were trying to make nsPresContext have the AnimationEventDispatcher as data object (not RefPtr<>) just like we did in CommonAnimationManager, we will fall into header inclusion hell since Element.h includes nsPresContext.h and AnimationEventDispatcher.h ends up including Element.h. Even if we could solve the inclusion hell, we will suffer from Rust bindgen issues for some reasons. MozReview-Commit-ID: B0nX2JzIRJD

Bug 1415780 - De-templatize AnimationEventDispatcher. r?birtles,masayuki Now single AnimationEventDispatcher can handle both CSS animation/transition events simultaneously. To do this we had to change AnimationEventInfo and TransitionEventInfo into a single struct, the struct is also named AnimationEventInfo. It results we can sort both CSS animation/transition events altogether. Thus we make sure CSS animation/transition events are sorted by scheduled event time prior to their composite order (i.e. transitions is prior to animations). At this moment, we don't sort both events altogether since nsAnimationManager and nsTransitionManager has an AnimationEventDispatcher respectively. In the next patch we move AnimationEventDispatcher into nsPresContext, i.e. each document has an AnimationEventDispatcher without the distinction between CSS animations and transitions. Note that, after this patch, we copy all members in InternalTransitionEvent and InternalAnimationEvent in the copy-constructor of AnimationEventInfo, this will be fixed once WidgetEvent has move-constructor and move-assignment (bug 1433008). MozReview-Commit-ID: 5JAh6N7C6ee

Bug 1415780 - Split AnimationEventDipatcher into an independent file. r?birtles MozReview-Commit-ID: Fcqtu7G400Z

Bug 1415780 - Rename DelayedEventsDispatcher to AnimationEventDispatcher. r?birtles We plan to use this class to dispatch web animation API events as well (bug 1354501). MozReview-Commit-ID: 7DMmc0aJzJ5

Bug 1415780 - Drop the comment for nsAnimationManager::DispatchEvents(). r?birtles The comment is no longer correct since bug 1180125. MozReview-Commit-ID: A4NWgmwAOFt

Bug 1415780 - Move AnimationCollection::PseudoTypeAsString into nsCSSPseudoElements. r?birtles It's not need to be a template function since the function is independent from AnimationType. MozReview-Commit-ID: 13WkRIfaQbZ

Bug 1415780 - Don't count all observers for nsRefreshDriver. r?smaug We just need information whether there is still an observer or not in most cases. The only case we need to know the count is in an assertion in the dtor of nsRefreshDriver. In the dtor we are checking there remains no observers other than early runners. Note that the order in HasObserver() was adjusted to reflect that we check boolean flag first (mViewManagerFlushIsPending) and subsequently check observer which is likey happen to. mFrameRequestCallbackDocs should have checked prior to mResizeEventFlushObservers though. MozReview-Commit-ID: E1qplusqw1Y

Bug 1415780 - Undef GetCurrentTime in the template function that uses Animation::GetCurrentTime. r? MozReview-Commit-ID: DaOZtYRx4Ud

No bug, Automated HPKP preload list update from host bld-linux64-spot-304 - a=hpkp-update