Closed Bug 1320289 Opened 7 years ago Closed 7 years ago

Register STS thread to gecko profiler

Categories

(Core :: Networking, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: schien, Assigned: schien)

Details

(Whiteboard: [necko-active])

Attachments

(1 file)

Need explicitly register nsThread to gecko profiler in order to view thread activity on Cleopatra.
Comment on attachment 8814350 [details]
Bug 1320289 - register STS thread to gecko profiler.

https://reviewboard.mozilla.org/r/95622/#review95672

Yes!  There is also possible to do:

profiler_register_thread(PR_GetThreadName(PR_GetCurrentThread()), ...);
Attachment #8814350 - Flags: review?(honzab.moz) → review+
(In reply to Honza Bambas (:mayhemer) from comment #2)
> Comment on attachment 8814350 [details]
> Bug 1320289 - register STS thread to gecko profiler.
> 
> https://reviewboard.mozilla.org/r/95622/#review95672
> 
> Yes!  There is also possible to do:
> 
> profiler_register_thread(PR_GetThreadName(PR_GetCurrentThread()), ...);

Done.
thanks.. this has been on my 'ought to do' list forever
Comment on attachment 8814350 [details]
Bug 1320289 - register STS thread to gecko profiler.

https://reviewboard.mozilla.org/r/95622/#review96298

LGTM. Sorry for the late response.
Attachment #8814350 - Flags: review?(cyu) → review+
Pushed by schien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/db8158d1b31d
register STS thread to gecko profiler. r=cyu,mayhemer
https://hg.mozilla.org/mozilla-central/rev/db8158d1b31d
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: