Open Bug 1827577 Opened 2 years ago Updated 2 years ago

Clean up the profiler old profiler panel related instrumentation code in C++ side

Categories

(Core :: Gecko Profiler, task, P3)

task

Tracking

()

Tracking Status
firefox114 --- affected

People

(Reporter: canova, Unassigned)

References

Details

With Bug 1668219, we removed the old performance panel for good. Before this, we had a way to switch between old/new with a preference. But now it's gone, we can remove all of the instrumentation code, related to markers etc.
Here are some examples (not exhaustive):
recordProfileTimelineMarkers
Probably the whole TimelineConsumers? (not sure if it's used outside of the old performance panel. we neeed to look, but I highly doubt it.)
But if we can't remove everything there, at least we should remove:
TimelineConsumers::AddMarkerForDocShell, TimelineConsumers::AddMarkerForAllObservedDocShells TimelineConsumers::PopMarkersand their usages.

Summary: Clean up the profiler old profiler panel related instrumentation code → Clean up the profiler old profiler panel related instrumentation code in C++ side
You need to log in before you can comment on or make changes to this bug.