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)
Firefox
General
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)
|
16.15 KB,
text/html
|
Details | |
|
7.08 KB,
patch
|
Details | Diff | Splinter Review |
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.
Updated•8 years ago
|
Updated•8 years ago
|
Flags: qe-verify?
Priority: -- → P2
Whiteboard: [photon-performance]
Updated•8 years ago
|
Blocks: photon-perf-jank
Updated•8 years ago
|
No longer blocks: photon-performance-triage
Comment 1•8 years ago
|
||
(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
Updated•8 years ago
|
Flags: qe-verify? → qe-verify-
| Assignee | ||
Updated•8 years ago
|
Assignee: nobody → felipc
Status: NEW → ASSIGNED
Updated•8 years ago
|
Priority: P2 → P1
Updated•8 years ago
|
Whiteboard: [photon-performance] → [reserve-photon-performance]
| Assignee | ||
Comment 2•8 years ago
|
||
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)
Keywords: stale-bug
| Reporter | ||
Comment 3•7 years ago
|
||
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
| Assignee | ||
Comment 4•7 years ago
|
||
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.
| Assignee | ||
Comment 5•7 years ago
|
||
(Note: the startup timers were fixed at bug 1388145)
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
You need to log in
before you can comment on or make changes to this bug.
Description
•