Closed Bug 1354723 Opened 8 years ago Closed 7 years ago

Understand what scripts in the browser front-end are run on timeouts and intervals

Categories

(Firefox :: General, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 57

People

(Reporter: mconley, Assigned: Felipe)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Whiteboard: [reserve-photon-performance])

Attachments

(2 files)

I was discussing this with ehsan today. Having looked at a number of profiles, we're concerned that there are timers and intervals being queued that might run at any time on the main thread. It's kind of the wild west. We have no idea how many of these things there are, we don't know their performance characteristics, and we don't know how often they run. We should probably get a sense of this, at least to put our minds a bit at ease.
Depends on: 1353584, 1353586
Flags: qe-verify?
Priority: -- → P2
Whiteboard: [photon-performance]
Depends on: 1357114
Depends on: 1357116
Depends on: 1357146
Depends on: 1357154
Depends on: 1360261
Depends on: 1360874
Depends on: 1361431
(In reply to Mike Conley (:mconley) from comment #0) > that might run at any time on the main thread. It's kind of the wild west. Bug 1365970 is about the blocking on _content process_, please remove this dependency if it's not for this meta bug.
Depends on: 1365970
Flags: qe-verify? → qe-verify-
Assignee: nobody → felipc
Status: NEW → ASSIGNED
Priority: P2 → P1
Whiteboard: [photon-performance] → [reserve-photon-performance]
Attached file settimeouts.htm
This is all the de-duplicated setTimeout calls that I got during a browsing session where I tried to open all the main browser functionalities (bookmarks, add-ons manager, menus, etc)
If anybody ever picks this up, njn just pointed me at this document, which seems super relevant: https://developer.mozilla.org/en-US/docs/Mozilla/Performance/TimerFirings_logging
All the work meant for photon startup perf was completed here. I meant to get back to this bug to investigate the non-startup timers, but that ship has sailed, and it's better handled in a new bug. I'm attaching the hacky patch that I had here to log these setTimeout calls, and will close this bug.
(Note: the startup timers were fixed at bug 1388145)
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Keywords: stale-bug
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: