1434649 - Push the worker script URL label frame even when the profiler is inactive

Status: NEW

(Core :: Gecko Profiler, defect, P2)

Description

[Markus Stange [:mstange]]

https://searchfox.org/mozilla-central/rev/c56f656febb1aa58c72b218699e24a76425bd284/dom/workers/RuntimeService.cpp#2840 :

> NS_IMETHODIMP
> WorkerThreadPrimaryRunnable::Run()
> {
>   AUTO_PROFILER_LABEL_DYNAMIC_LOSSY_NSSTRING(
>     "WorkerThreadPrimaryRunnable::Run", OTHER, mWorkerPrivate->ScriptURL());

AUTO_PROFILER_LABEL_DYNAMIC_LOSSY_NSSTRING only pushes an entry if the profiler is running at the time that this function is entered. That makes sense for functions that get called frequently and only have a short runtime, but it's not great for long-running functions like the DOM worker's primary thread runnable, which stays on the stack for the entire duration that that script is run on the worker thread. So if the worker thread starts before the profiler is started, its script URL will not show up in profiles.