1609674 - Add innerWindowID to "setTimeout" and "setTimeout deferred release" markers

Status: RESOLVED FIXED

(Core :: Gecko Profiler, enhancement)

Description

[Nazım Can Altınova [:canova][:canaltinova on phabricator]]

We don't include innerWindowID for those markers as of now. But there is a mWindow field inside TimeoutManager class, that means we can actually get that ID for those markers. We should definitely add those.

We have innerWindowID for "setTimeout callback" marker but even though we can get that information easily, with WindowID(), we go through docshell -> outerWindow -> innerWindow -> WindowID(). We should definitely fix that too while we are here.

Comment 1

[Nazım Can Altınova [:canova][:canaltinova on phabricator]]

Attachment: Bug 1609674 - Add innerWindowID to setTimout markers. r?gerald

Comment 2

[Nazım Can Altınova [:canova][:canaltinova on phabricator]]

Attachment: Bug 1609674 - Add innerWindowID param to AUTO_PROFILER_TEXT_MARKER_CAUSE and use that macro for setTimeout callback marker. r?gerald

Depends on D60269

Comment 3

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/fd633beac3d4
https://hg.mozilla.org/mozilla-central/rev/4b4d97f009d3