1757202 - Add more information in profiler markers showing CSS animations and transitions

Status: RESOLVED FIXED

(Core :: CSS Transitions and Animations, task)

Description

[Florian Quèze [:florian]]

It's currently difficult to understand which animations are running on the compositor

Comment 1

[Florian Quèze [:florian]]

Example profile with the proposed changes: https://share.firefox.dev/3M3Nrjg

Comment 2

[Florian Quèze [:florian]]

Attachment: Bug 1757202 - Make the inner window id of the browser window available in the compositor bridge, r=mstange.

Comment 3

[Florian Quèze [:florian]]

Attachment: Bug 1757202 - Include the inner window id in the existing Composite markers, r=mstange.

Depends on D139730

Comment 4

[Florian Quèze [:florian]]

Attachment: Bug 1757202 - Expose CompositorThreadHolder::GetThreadId to make it easy to add profiler markers on the compositor thread from any thread, r=mstange.

Depends on D139731

Comment 5

[Florian Quèze [:florian]]

Attachment: Bug 1757202 - Add SetAnimation, SampleAnimation and ClearAnimation markers on the compositor thread, r=mstange.

Depends on D139732

Comment 6

[Florian Quèze [:florian]]

Attachment: Bug 1757202 - Add more information in existing CSS Animation and CSS Transition main thread markers, r=mstange,boris.

Depends on D139733

Comment 7

[Florian Quèze [:florian]]

Attachment: Bug 1757202 - Add the browser window inner id to the SetDisplayList, CompositeToTarget and SkippedComposite profiler markers, r=mstange.

Depends on D139733

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3d7c412ba08d
Add more information in existing CSS Animation and CSS Transition main thread markers, r=boris.
https://hg.mozilla.org/integration/autoland/rev/52606df2ea3d
Make the inner window id of the browser window available in the compositor bridge, r=mstange.
https://hg.mozilla.org/integration/autoland/rev/4601c9c04d7b
Include the inner window id in the existing Composite markers, r=mstange.
https://hg.mozilla.org/integration/autoland/rev/b989b20e9704
Expose CompositorThreadHolder::GetThreadId to make it easy to add profiler markers on the compositor thread from any thread, r=mstange.
https://hg.mozilla.org/integration/autoland/rev/1553b2367135
Add SetAnimation, SampleAnimation and ClearAnimation markers on the compositor thread, r=mstange.
https://hg.mozilla.org/integration/autoland/rev/da54c46e2e6d
Add the browser window inner id to the SetDisplayList, CompositeToTarget and SkippedComposite profiler markers, r=mstange.

Comment 9

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/3d7c412ba08d
https://hg.mozilla.org/mozilla-central/rev/52606df2ea3d
https://hg.mozilla.org/mozilla-central/rev/4601c9c04d7b
https://hg.mozilla.org/mozilla-central/rev/b989b20e9704
https://hg.mozilla.org/mozilla-central/rev/1553b2367135
https://hg.mozilla.org/mozilla-central/rev/da54c46e2e6d