Closed Bug 613260 Opened 15 years ago Closed 15 years ago

Determine Header Performance Impact

Categories

(www.mozilla.org :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: lforrest, Assigned: sgarrity)

References

()

Details

As discussed, investigate performance ramifications of the header on download/ie page. There's no java involved, just CSS, so it's unclear how much load time it's adding.
As Laura mentioned, the header flydown navigation menus do no use Javascript except for a *tiny* (less than 1Kb) function that is used by IE6 only. We could load this only for IE6 as well, to further minimize the already small impact on other visitors. The CSS for the menu comprises about 15% of the template.css file. This file is about 14Kb, but is served gzipped at around 3Kb, so that 15% gets to be pretty minimal. Point being, I doubt that the header menu feature has a significant impact on performance or downloads. I'm content to be proven wrong with data thought. Alex, anything I'm missing on this?
(In reply to comment #1) > Alex, anything I'm missing on this? Makes sense to me. Unless the HTTP request count is different per browser? In Firefox I have 17 requests, in IE 6-8 I see 21. http://www.webpagetest.org/result/101118_D0ZT/1/details/
(In reply to comment #2) > Unless the HTTP request count is different per browser? In > Firefox I have 17 requests, in IE 6-8 I see 21. I think this is due to a WOFF 404 that IE gets - I need to port over Anthony's fix for that from the A/B test page.
I think we shouldn't remove the header menu for performance reasons. As Steven said, the code needed is insignificant. There are many other things to do to improve performance on this page.
(In reply to comment #4) > I think we shouldn't remove the header menu for performance reasons. As Steven > said, the code needed is insignificant. There are many other things to do to > improve performance on this page. Is it worth making this change for the test in bug 612741, then? Blake, unless you have objections I'd like to give it a try.
(In reply to comment #5) > Is it worth making this change for the test in bug 612741, then? Blake, unless > you have objections I'd like to give it a try. Which change are you talking about? If it's the extra WOFF 404 request Steven talked about, it is already fixed in the tests. If you're talking about "many other things to do to improve performance on this page." that I mentioned, they are not worth it on the test pages.
(In reply to comment #6) > Which change are you talking about? Sorry - I meant the general "making the header pulldowns work again" topic. If that's taken care of by the extra WOFF 404 request, then nevermind.
Rather than speculating, I suggest we run a test and measure the actual impact. In addition to the performance impact, I'm curious how increasing the number of outbound links from this page will affect conversions.
(In reply to comment #8) > Rather than speculating, I suggest we run a test and measure the actual impact. > In addition to the performance impact, I'm curious how increasing the number > of outbound links from this page will affect conversions. Agreed...would be great to test this. Any chance we can make this happen concurrently with what's being tested by bug 612741? We need to know this info pretty soon.
yes, I just need a second version of that page built with the full header.
(In reply to comment #10) > yes, I just need a second version of that page built with the full header. Alex or Anthony, is that something you guys could help with this afternoon?
(In reply to comment #11) > (In reply to comment #10) > > yes, I just need a second version of that page built with the full header. > > Alex or Anthony, is that something you guys could help with this afternoon? Yes. Could you clarify exactly what you want? Which page am I copying? I'm just adding the header drop-down menus?
Please copy this page: http://www-trunk.stage.mozilla.com/en-US/nova-test-596010/upgrade.html, and only add the drop down menu.
http://www-trunk.stage.mozilla.com/en-US/nova-test-596010/header Adding nova-test-596010/header.html Transmitting file data . Committed revision 77862.
Thanks Alex!
Blocks: 613992
Filled bug 613992 to specifically track the header tests.
No longer blocks: 613992
Depends on: 613992
Can I get this pushed? Ready to test once it's live.
Adding tags/production/en-US/nova-test-596010/header.html Adding tags/stage/en-US/nova-test-596010/header.html Committed revision 78048.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Component: www.mozilla.org/firefox → www.mozilla.org
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in before you can comment on or make changes to this bug.