Open
Bug 341046
Opened 18 years ago
Updated 2 years ago
Incremental layout should not happen in background tabs
Categories
(Core :: Layout, enhancement)
Core
Layout
Tracking
()
NEW
People
(Reporter: aguertin+bugzilla, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: perf)
Incremental layout displays some of a web page's content faster at the expense of slowing down the loading process as a whole. In foreground tabs this is good, as it speeds up getting information, but in background tabs it's pointless, and just wastes resources.
This is especially bad when opening many tabs at once, or several large, table-heavy tabs (e.g. phpBB forum threads). It has contributed to things like bug 333734.
Disabling incremental layout on background tabs would be a big help.
Note that this needs to be fleshed out more. E.g. a user is more likely to switch to first or last tab, so do incremental layout on those? Perhaps half as often on those?
Comment 1•18 years ago
|
||
Certainly speaks to bug 333734 comment 2, and sounds quite sensible. Do we have a sense of what sort of performance improvement this would yield?
Comment 2•18 years ago
|
||
david / bryner, in addition to not doing incremental layout (for a tab loading in the background) are there any other things we could not do for background tabs?
Comment 3•18 years ago
|
||
I don't know about not doing other things for background tabs, but giving network priority to the currently focused tab would help load times too (bug 305402)
Comment 4•18 years ago
|
||
brendan, this was the bug i mentioned that might be able to help the "several dojo apps in different tabs" performance issue.
Comment 5•18 years ago
|
||
on a related note, see 336060
Comment 6•17 years ago
|
||
brad, this is the bug I mentioned in today's meeting.
Comment 7•17 years ago
|
||
Noming for 1.9 since improving performance/responsiveness with multiple tabs open is critical.
Flags: blocking1.9?
Comment 8•17 years ago
|
||
I'd note that there's a tradeoff between amount of work done while things are in the background and the response time when switching them back to the foreground. But I suspect we might be able to come up with a good compromise -- probably one involving reducing the priority / frequency of updates for things in background tabs but still not stopping them completely.
Flags: blocking1.9? → blocking1.9-
Whiteboard: [wanted-1.9]
Updated•17 years ago
|
Flags: wanted1.9+
Whiteboard: [wanted-1.9]
Comment 9•17 years ago
|
||
I guess this is critical for session restore, too.
Comment 10•17 years ago
|
||
(In reply to comment #9)
> I guess this is critical for session restore, too.
>
Why?
Comment 11•17 years ago
|
||
I'm assuming because if you've got a lot of tabs being restored, it's possible that the browser could grind to a halt trying to render everything at the same time.
Comment 12•17 years ago
|
||
I can reproduce such behavior with current trunk on my linux box. Restoring a session freezes the browser for a ~10-15 seconds. (same happens when I start Firefox and try to write into address bar - I think it's because Firefox is trying to rebuild the smartbar autocomplete)
This would be really great.
Assignee: nobody → roc
Flags: wanted1.9-
Flags: wanted1.9+
Flags: wanted-next+
Comment 14•16 years ago
|
||
> but in background tabs it's pointless, and just wastes resources.
One thing about incremental layout is that it _does_ prevent long pauses (which would otherwise happen when the whole layout is done in one shot at the end of the page load). So it's not quite pointless even in background tabs, though I agree that rendering the content in somewhat larger chunks is almost certainly desirable.
Updated•16 years ago
|
Flags: wanted1.9.2?
Reporter | ||
Comment 15•16 years ago
|
||
One thing I just realized about this is that getting the page titles into tabs as quickly as possible *is* important, because those are always visible. So this should take steps (if needed) not to break that.
Flags: wanted1.9.2?
Flags: wanted1.9.2-
Flags: blocking1.9.2-
Assignee: roc → nobody
Updated•13 years ago
|
Whiteboard: [Snappy]
Comment 16•13 years ago
|
||
We should look into suspending/throttling background reflows, switching/turning off incremental reflows risks introducing longer pauses.
Whiteboard: [Snappy]
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•