Closed Bug 1641298 Opened 4 years ago Closed 3 years ago

Screenshots are shown for closed windows

Categories

(Core :: Gecko Profiler, defect, P2)

defect

Tracking

()

RESOLVED FIXED
92 Branch
Tracking Status
firefox92 --- fixed

People

(Reporter: florian, Assigned: florian)

References

Details

Attachments

(1 file)

The profiler doesn't seem to record when a window is closed, so the profiler front-end keeps on showing the last screenshot of windows that have been closed.

The CompositorScreenshot markers have a windowID field that contains a pointer to the window object, but the DOMWindowClosed DOMEvent markers have a innerWindowID field that's an integer.

It's not a pointer to the window object, it's a pointer to "something that only exists once per window". For example for the non-WebRender screenshots implementation, it's the address of the CompositorScreenshotGrabberImpl object.
So I think there are two ways to fix this: Either we carry the innerWindowID field all the way to the compositor (which might be tricky), or we emit CompositorScreenshotWindowDestroyed markers from the compositor screenshot infrastructure, for example from the CompositorScreenshotGrabberImpl destructor, with an appropriate windowID that matches the CompositorScreenshot markers.

The severity field is not set for this bug.
:gerald, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(gsquelart)
Severity: -- → S3
Flags: needinfo?(gsquelart)
Assignee: nobody → florian
Status: NEW → ASSIGNED
Pushed by fqueze@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/960a24445eee
record a CompositorScreenshotWindowDestroyed marker from ProfilerScreenshots to indicate to the profiler front-end that a window has been closed, r=mstange.
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: