Thread names of the content process are lost

RESOLVED FIXED in Firefox 53

Status

()

Product:
Core
Component:
Gecko Profiler
Type:
defect
Status:
RESOLVED FIXED
Opened:
3 years ago
Updated:
3 years ago

People

(Reporter: mstange, Unassigned)

Tracking

Version:
Trunk
Target:
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

Bug 1322471 - Report the true thread name for content process threads in the profile, and add a processType field.
58 bytes, text/x-review-board-request
BenWa
: review+
Details 
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 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+
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
https://hg.mozilla.org/mozilla-central/rev/833fef6e2700
Status: NEW → RESOLVED
Closed: 3 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.