try_task_config for https://phabricator.services.mozilla.com/D80740 Differential Diff: PHID-DIFF-q3kbjvsvnqbiloxfm5b2

Bug 1647342 - Part 7: Remove unnecessary special-casing of OOM in replaceFrameGuts. r=arai! Summary: This patch waits to remove the frame from the "from" map until after it has been added to the "to" map, so that we can rely on the existing scope guard to clean up any frames if we get an OOM. Differential Revision: https://phabricator.services.mozilla.com/D80740 Depends on D80739 Test Plan: Reviewers: arai! Subscribers: Bug #: 1647342 Differential Diff: PHID-DIFF-q3kbjvsvnqbiloxfm5b2

Bug 1647342 - Part 6: Simplfy code in onResumeFrame. r=arai! Summary: There's no reason to create this FrameIter over and over again inside the loop. Similarly, ensureExecutionObservabilityOfFrame is a no-op if "frame.isDebuggee()", and slowPathOnResumeFrame asserts that to be true, so there is no reason to ensure observability here because it already is guaranteed. Differential Revision: https://phabricator.services.mozilla.com/D80739 Depends on D80738 Test Plan: Reviewers: arai! Subscribers: Bug #: 1647342 Differential Diff: PHID-DIFF-vtmty3jilj4hdtecpioc

Bug 1647342 - Part 5: Move management of the generatorFrames collection. r=arai! Summary: It is difficult to follow the overall lifecycle of entries in the generatorFrames list when it is mutated in many places across many files. This patch's primary aim is to begin centralizing the handling of this map a little bit by keeping it all inside Debugger.cpp so that the DebuggerFrame instance itself does not actually care whether it is found in that map. Differential Revision: https://phabricator.services.mozilla.com/D80738 Depends on D80737 Test Plan: Reviewers: arai! Subscribers: Bug #: 1647342 Differential Diff: PHID-DIFF-4mkpmb6bbdnaebxdfjm3

Bug 1647342 - Part 4: Convert forEachDebuggerFrame helper to expose Debugger*. r=arai! Summary: This is just a tiny refactor to avoid needing to then get the Debugger* from the frame when we already know it. Differential Revision: https://phabricator.services.mozilla.com/D80737 Depends on D80736 Test Plan: Reviewers: arai! Subscribers: Bug #: 1647342 Differential Diff: PHID-DIFF-a6x6lzo4hgfkgtlxayka

Bug 1647342 - Part 3: Remove unused detachAllDebuggersFromGlobal Debugger method. r=arai! Summary: I have no idea how long this has been stale. Differential Revision: https://phabricator.services.mozilla.com/D80736 Depends on D80735 Test Plan: Reviewers: arai! Subscribers: Bug #: 1647342 Differential Diff: PHID-DIFF-5mb2ccxp5m5n24dvyva3

Bug 1647342 - Part 2: Rely on onResumeFrame to handle marking all frames as on-stack. r=arai! Summary: It is guaranteed that this will already be populated because in 1539654 we started properly preserving the debuggee-ness of frames across suspend and resume. With slowPathOnResumeFrame guaranteed to run, we no longer need to try to hande handle the case where the generator has been resumed, but the Debugger.Frame's inner data was not populated. Differential Revision: https://phabricator.services.mozilla.com/D80735 Depends on D80734 Test Plan: Reviewers: arai! Subscribers: Bug #: 1647342 Differential Diff: PHID-DIFF-zb7v2adok4h6dtiv25lb

Bug 1647342 - Part 1: Remove FrameIter::Data from DebuggerFrame public API. r=arai! Summary: Exposing frameIterData() and setFrameIterData() isn't great since it makes it harder to follow what state transitions for frame data we expect the frame to have. By hiding this, it helps clarify that frame data can be cleared or replaced, but public usage of the FrameIter::Data object itself isn't expected outside of the class itself. Differential Revision: https://phabricator.services.mozilla.com/D80734 Depends on D80762 Test Plan: Reviewers: arai! Subscribers: Bug #: 1647342 Differential Diff: PHID-DIFF-dhmfmqn66xpowicdvjn7

Bug 1577639 - Do not attempt to clean up frames for non-OOM conditions. r=arai! Summary: The core issue is that we run removeFromFrameMapsAndClearBreakpointsIn(cx, from) even for non-OOM paths in this function, which is undesirable because that will remove the breakpoints for evaled scripts. In the non-OOM case there are no Debugger.Frame objects left in the frames map for the "frame" list, so there is no reason to actually call it in that case. This patch merges the two OOM handlers together for simplicity, though it is not technically necessary, it seems like the very minor increase in code run if there is an OOM is worth it to keep things centralized. Differential Revision: https://phabricator.services.mozilla.com/D80762 Test Plan: Reviewers: arai! Subscribers: Bug #: 1577639 Differential Diff: PHID-DIFF-fh4vivxlaoteqp2ezgnf

Tasks automatically selected. Pushed via `mach try auto`

Bug 1324591 - Disable partial pre-render for frames in position:sticky subtree. r?botond Differential Revision: https://phabricator.services.mozilla.com/D77774

Bug 1324591 - Report janked animations to the main-thread and update them on the main-thread. r?botond!,boris! The machinery to report janked animations is; 1) Store the partial pre-rendered animation id and the Animation object in a hashtable in LayerManager 2) Store the animation id in the Animation object as well 3) When we detect jank, we send the animation id to the main-thread via an IPC call 4) Find the Animation object with the id in the hashtable and update the Animaiton 5) Whenever the partial pre-rendered Animation stop running on the compositor i.e. the Animation finished normally, the Animation's target element is changed, etc. etc., remove the Animation from the hashtable Differential Revision: https://phabricator.services.mozilla.com/D75732

Bug 1324591 - Jank if we are trying to composite area which is outside of the the partial prerender rect on non WebRender. r?botond Differential Revision: https://phabricator.services.mozilla.com/D75731

Bug 1324591 - Inform the partial prerender rect and the overflowed SideBits to the compositor. r?botond nsDisplayTransform doesn't have enough room to store the partial prerender rect and the overflowed SideBits unfortunately (it will exceed 512 bytes limit [1]). So we get the partial prerender rect and calculate the overflowed SideBits just before we send transform animation information to the compositor. [1] https://searchfox.org/mozilla-central/rev/fa52bedc4b401c12251513fa1c9df1753a29abb2/layout/painting/nsDisplayList.cpp#7318 Differential Revision: https://phabricator.services.mozilla.com/D75730

Bug 1324591 - Store PrerenderDecision in nsDisplayTransform. r?botond Differential Revision: https://phabricator.services.mozilla.com/D75729

Bug 1324591 - Store the corresponding LayersId for each animation on the compositor. r?kats Differential Revision: https://phabricator.services.mozilla.com/D75728

Bug 1324591 - A simple mochitest to make sure partial pre-rendered transform animations run on the compositor. r?boris We just need this regardless of whether there appears checkerboarding or jank. Differential Revision: https://phabricator.services.mozilla.com/D75727

Bug 1646629 - Set updateCanvasPending explicitly when reftest-wait was removed and take a snapshot of the whole screen if reftest-no-flush is specified. r?mattwoodrow,tnikkel That's because we no longer fire MozAfterPaint event for changes by animations on the compositor. Differential Revision: https://phabricator.services.mozilla.com/D80157

try: -b o -p linux64-valgrind -t none -u none

try_task_config for https://phabricator.services.mozilla.com/D80739 Differential Diff: PHID-DIFF-vtmty3jilj4hdtecpioc