Thread names of the content process are lost

RESOLVED FIXED in Firefox 53

Status

()

Core
Gecko Profiler
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mstange, Unassigned)

Tracking

Trunk
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
http://searchfox.org/mozilla-central/source/tools/profiler/core/ThreadProfile.cpp#118 overwrites all threads in the content process with the name "Content". So if you profile e.g. worker threads or image decoder threads in the content process, they'll all be called "Content".
Comment hidden (mozreview-request)

Comment 2

2 years ago
mozreview-review
Comment on attachment 8817365 [details]
Bug 1322471 - Report the true thread name for content process threads in the profile, and add a processType field.

https://reviewboard.mozilla.org/r/97664/#review98020
Attachment #8817365 - Flags: review?(b56girard) → review+

Comment 3

2 years ago
Pushed by mstange@themasta.com:
https://hg.mozilla.org/integration/autoland/rev/833fef6e2700
Report the true thread name for content process threads in the profile, and add a processType field. r=BenWa

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/833fef6e2700
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.