1843262 - Add logging presets to profiler

Status: RESOLVED WORKSFORME

(Core :: Networking, task, P2)

Description

[Kershaw Chang [:kershaw]]

We've added logging prefets in bug 1838211, but it only works for files.
It'd be great if we can make gecko profiler also use these presets.

Comment 1

[Dylan]

Attachment: Bug 1843262 - Added presets for cookie, websocket, and Http3 to profiler. r=kershaw

Comment 2

[Julien Wajsberg [:julienw]]

Hey Dylan, Kershaw,
Instead of adding new profiler presets, you can instruct your about:logging presets to use the already existing "Networking" preset, by specifying profilerPreset: "networking" for this about:logging presets. Would that work well enough for you?

That would also prevent the cluttering of our main preset selection.

Comment 3

[Kershaw Chang [:kershaw]]

(In reply to Julien Wajsberg [:julienw] from comment #2)

Hey Dylan, Kershaw,
Instead of adding new profiler presets, you can instruct your about:logging presets to use the already existing "Networking" preset, by specifying profilerPreset: "networking" for this about:logging presets. Would that work well enough for you?

I believe this should be enough, but as I am not completely familiar with the profiler code, I am not certain. I will leave it to Dylan to determine the best approach. Thank you.

Comment 4

[Dylan]

I agree with Kershaw, that seems to be enough thanks!