Open Bug 1329120 Opened 8 years ago Updated 3 years ago

Rename GetProfile to GetSingleProcessProfile

Categories

(Core :: Gecko Profiler, task, P3)

task

Tracking

()

Tracking Status
firefox53 --- affected

People

(Reporter: mstange, Unassigned)

References

(Blocks 1 open bug)

Details

This method is a legacy method whose return value does not contain subprocess profiles. Ideally this method should be removed, but the devtools profiler calls it. It calls it from inside the content process though, and it's only interested in the content process profile, so that's mostly ok.

The name + presence of this method can confuse people who may expect it to include subprocess profiles.

Functions getProfilerData... now return a promise that will retrieve profiles from all processes.
There is no getProfileData anymore, but we still have GetProfile that returns the current process profile as a string. So it would still be relevant to mention "SingleProcess" in the function name.

Severity: normal → N/A
Type: defect → task
Priority: -- → P3
Summary: Rename getProfileData to getSingleProcessProfileData → Rename GetProfile to GetSingleProcessProfile
You need to log in before you can comment on or make changes to this bug.