Closed
Bug 1048843
Opened 10 years ago
Closed 10 years ago
[Messages] Investigate about concatenating/minifying CSS
Categories
(Firefox OS Graveyard :: Gaia::SMS, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
2.1 S2 (15aug)
People
(Reporter: julienw, Assigned: steveck)
References
Details
(Whiteboard: [p=1])
Attachments
(1 file)
In this bug we want to see if having several CSS files has an impact at launch time (as compared to one concatenated CSS). If this has an impact, we'll do a build-time optimization.
Reporter | ||
Updated•10 years ago
|
Target Milestone: --- → 2.1 S2 (15aug)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → schung
Assignee | ||
Comment 1•10 years ago
|
||
Manual testing with YUI compressor(minify + concatenate) result based on light workload/flame/master build: average time is 526ms(compressed) : 564ms(without compress). Applying the compressor only reduce ~40ms, seems not an impressive result... :-/ So minifying CSS might not be the most important issue for performance improvement. BTW, the load time without loading any css is 464ms, maybe we should also focus on fine-tune the css selector part(like shorten the long selector) for performance improvement in the future.
Reporter | ||
Comment 2•10 years ago
|
||
tentative patch to minify CSS Not sure at all this works properly yet.
Reporter | ||
Comment 3•10 years ago
|
||
Comment on attachment 8469293 [details]
github PR
Hey Yuren,
So, this seems to work fine on all applications. What do you think?
I don't see a very noticeable improvement in load time, using my own eyes, but I didn't try using the perf tools. Eli, maybe you can help testing this?
Attachment #8469293 -
Flags: feedback?(yurenju.mozilla)
Attachment #8469293 -
Flags: feedback?(eperelman)
Updated•10 years ago
|
Comment 4•10 years ago
|
||
We've done some CSS minify experiment a year ago. But after packaging, the result of loadtime improvement is not that noticeable :(
Comment 5•10 years ago
|
||
Comment on attachment 8469293 [details] github PR pull request looks good but I recommand to do more performance measurement before we implement it. we have implemented css minify on bug 855989 and don't have impressive improvement. so it would be great to use other css minify tool for some app and see if it's worth to implement.
Attachment #8469293 -
Flags: feedback?(yurenju.mozilla) → feedback+
Comment 6•10 years ago
|
||
Comment on attachment 8469293 [details]
github PR
Using:
make reference-workload-light
Against master:
"mozPerfDurationsAverage": 1906.1065674
Against master with GAIA_OPTIMIZE=1:
"mozPerfDurationsAverage": 1838.8873350333333
Against master+patch with GAIA_OPTIMIZE=1:
"mozPerfDurationsAverage": 1876.0379218
Attachment #8469293 -
Flags: feedback?(eperelman)
Reporter | ||
Comment 7•10 years ago
|
||
Thanks Eli, then it's quite clear it doesn't change anything, will close this bug then.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•