1862557 - Add profiler LongTask markers for SocketTransportService/Socket Thread

Status: RESOLVED FIXED

(Core :: Networking, enhancement, P3)

Description

[Randell Jesup [:jesup] (needinfo me)]

Note due to the Poll usage we have to add a second marker injection to record time spent calling OnSocketReady() for any woken-up sockets.

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

Attachment: Bug 1862557: Add LongTask profiler marker generation to SocketThread r=#necko-reviewers!

Includes separate markers for OnSocketReady() calls after Poll()

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4ef47a21279a
Add LongTask profiler marker generation to SocketThread r=necko-reviewers,kershaw

Comment 3

[Cristian Tuns]

Backed out for causing build bustages in nsThreadManager.cpp

• Backout link
• Push with failures
• Failure Log
• Failure line: /builds/worker/checkouts/gecko/xpcom/threads/nsThreadManager.cpp:313:75: error: no matching function for call to 'nsThread::nsThread(mozilla::NotNull<RefPtr<mozilla::ThreadEventQueue> >, nsThread::MainThreadFlag, <brace-enclosed initializer list>)'

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/93a04e8835aa
Add LongTask profiler marker generation to SocketThread r=necko-reviewers,kershaw

Comment 5

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/93a04e8835aa