Closed Bug 1322471 Opened 4 years ago Closed 4 years ago

Thread names of the content process are lost

Categories

(Core :: Gecko Profiler, defect)

Product:
Core
Component:
Gecko Profiler
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla53
Tracking Flags:
Tracking Status
firefox53 --- fixed

People

(Reporter: mstange, Unassigned)

Details

Attachments

(1 file)

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: 4 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.