Closed Bug 1772284 Opened 3 years ago Closed 3 years ago

ImageLoader::Notify profiler label creates extra variables on the stack

Categories

(Core :: Layout: Images, Video, and HTML Frames, defect)

defect

Tracking

()

RESOLVED FIXED
103 Branch
Tracking Status
firefox103 --- fixed

People

(Reporter: tnikkel, Assigned: tnikkel)

References

Details

Attachments

(1 file)

Apparently this is because of the conditional on the profiler being active here https://searchfox.org/mozilla-central/rev/c8e15e17bc6fd28f558c395c948a6251b38774ff/tools/profiler/public/ProfilerLabels.h#110,113 and bug 1771776 tracks that.

But we can avoid that by just using AUTO_PROFILER_LABEL_DYNAMIC_CSTR

Apparently this is because of the conditional on the profiler being active here https://searchfox.org/mozilla-central/rev/c8e15e17bc6fd28f558c395c948a6251b38774ff/tools/profiler/public/ProfilerLabels.h#110,113 and bug 1771776 tracks that.

But we can avoid that by just using AUTO_PROFILER_LABEL_DYNAMIC_CSTR

Assignee: nobody → tnikkel
Status: NEW → ASSIGNED
Summary: ImageLoader::Notify profiler label doesn't appear → ImageLoader::Notify profiler label creates extra variables on the stack
Attachment #9279384 - Attachment description: Bug 1772284. Make ImageLoader::Notify profiler label show up. r?emilio,gerald → Bug 1772284. Make ImageLoader::Notify profiler label not create extra variables on the stack. r?emilio,gerald
Pushed by tnikkel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/df37233057ab Make ImageLoader::Notify profiler label not create extra variables on the stack. r=emilio
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: