Closed
Bug 1203464
Opened 9 years ago
Closed 8 years ago
[Messages][Performance] Consider delaying of expensive gaia-header initialization for the secondary views
Categories
(Firefox OS Graveyard :: Gaia::SMS, defect, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: azasypkin, Unassigned)
References
Details
(Keywords: perf)
From profiles in bug 1198266, we see that initialization of gaia-header can be quite expensive (~20ms for every gaia-header on Flame-KK). For non-split views we initialize 6 gaia-headers while only one is really needed to be displayed as soon as possible. It also seems that we remove "no-font-fit" from secondary edit mode gaia-header too early: in InboxView.beforeEnter and ConversationView.beforeEnter.
So I this bug we want to try the following things:
* Add/initialize secondary gaia-headers only when needed or at least when "visuallyLoaded" is fired;
* Delay removing of "no-font-fit" for the edit mode header.
Reporter | ||
Updated•9 years ago
|
Updated•9 years ago
|
Reporter | ||
Comment 1•9 years ago
|
||
In the scope of bug 1211395, Steve fixed "edit" gaia-headers issues, so the only thing we want to consider here is to lazily insert gaia-headers to the DOM for the following cases to spend less time on gaia-header web component initialization: edit headers and headers for Report and Group views.
However we need to retake profiles first to see if it still can be significant issue + this also can be "automatically" fixed once we split these views.
Updated•9 years ago
|
Priority: -- → P3
Comment 2•8 years ago
|
||
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Comment 3•8 years ago
|
||
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in
before you can comment on or make changes to this bug.
Description
•