Closed
Bug 599852
Opened 14 years ago
Closed 14 years ago
initial group switch is very slow (nothing for seconds)
Categories
(Firefox Graveyard :: Panorama, defect, P1)
Tracking
(blocking2.0 final+)
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | final+ |
People
(Reporter: dietrich, Assigned: seanedunn)
References
Details
(Keywords: perf, Whiteboard: [tsnap])
Attachments
(1 file)
4.89 KB,
patch
|
Details | Diff | Splinter Review |
if i switch groups using the keyboard shortcut (ctrl/cmd + ~) without ever going into tabcandy, the first switch is extremely slow to happen.
sometimes it'll take up to 5 seconds.
Updated•14 years ago
|
blocking2.0: --- → ?
Comment 1•14 years ago
|
||
Note that the fix for this will likely be related to the fix for bug 595020.
Assignee: nobody → ian
Priority: -- → P2
Updated•14 years ago
|
Priority: P2 → P1
Comment 2•14 years ago
|
||
The time in question is the Panorama start-up time (as it needs to start up in order to get the group switch info). Looks like bug 588630 is going to make a big difference here. On my machine, without that patch, starting up Panorama with 100 tabs fully loaded takes 5 seconds, and 200 tabs takes 15 seconds. With the patch, 200 tabs takes 2 seconds.
Depends on: 588630
Comment 3•14 years ago
|
||
Beyond that, we could potentially load up the data and intelligence of Panorama without loading up the UI, since in this case the UI isn't needed. This will happen naturally as a part of bug 578512 (post ff4.0). Prior to that it's probably too radical of an architecture change. We should simply focus continuing to improve Panorama start time in general.
Making this bug dependent on bug 588630; this can be closed when it lands, or marked as duplicate before.
Assignee: ian → seanedunn
Comment 4•14 years ago
|
||
I'm attaching the profiling test I used for this bug. Note that it has two "failures"; that's how I report how long it took to initialize Panorama (relevant to this bug) and show Panorama (just of interest, but not relevant to the bug). Change the 200 to some other number to load more or fewer tabs.
Reporter | ||
Updated•14 years ago
|
Whiteboard: [tsnap]
Comment 5•14 years ago
|
||
Closing this now that bug 588630 has landed.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 6•14 years ago
|
||
Posthumous blocking+.
Status: RESOLVED → REOPENED
blocking2.0: ? → final+
Resolution: FIXED → ---
Reporter | ||
Comment 7•14 years ago
|
||
Ugh, flag munging.
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•