Closed Bug 1134602 Opened 9 years ago Closed 9 years ago

Fix dns prefetch for google analytics

Categories

(Marketplace Graveyard :: Consumer Pages, defect, P3)

Product:
Marketplace Graveyard
Component:
Consumer Pages
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
2015-03-17

People

(Reporter: mat, Assigned: kngo)

References

Details

(Keywords: perf, Whiteboard: [goodfirstbug]) 

In fireplace's src/app.html we have:
<link rel="dns-prefetch" href="//ssl.google-analytics.com">

This is good, except... :
- tracking.js uses https://www.google-analytics.com/analytics.js
- that script uses https://www.google-analytics.com/analytics and https://stats.g.doubleclick.net/
- zamboni's mkt/commonplace/templates/commonplace/index.html, which is the template actually used in production, only sets <link rel="dns-prefetch" href="{{ media_origin }}">


dns-prefetching our media origin is probably useless (the other <link> elements in our <head> already references that origin), so we should fix zamboni to properly prefetch //www.google-analytics.com and maybe //stats.g.doubleclick.net as well. It's trivial but could help perf on mobile networks, where latency is killer.
Priority: -- → P3
Whiteboard: [goodfirstbug]
Unsure if this is still relevant (probably), but adding to UA tracker.
Assignee: nobody → kngo
Blocks: 1134407
https://github.com/mozilla/zamboni/pull/2975
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2015-03-17
Please add some STRs to this bug or mark it as [qa-]
You need to log in before you can comment on or make changes to this bug.