#37385 closed defect (bug) (fixed)
Resource hinting: only dns-prefetch enqueued resources.
Reported by: | peterwilsoncc | Owned by: | peterwilsoncc |
---|---|---|---|
Milestone: | 4.6 | Priority: | normal |
Severity: | normal | Version: | 4.6 |
Component: | Script Loader | Keywords: | commit |
Focuses: | Cc: |
Description
dns-prefetch link elements are currently created for all registered depenedencies, only create them for enqueued dependencies.
Attachments (1)
Change History (8)
#2
@
8 years ago
@swissspidy there is in #34292 but forking it because it's become a bit of a super ticket.
I've brought the code across in 37385.diff, props due are code: @swissspidy, tests: @peterwilsoncc
#6
follow-up:
↓ 7
@
8 years ago
Should the function be called wp_get_dependencies_unique_hosts()
as suggested in 50:ticket:34292?
#7
in reply to:
↑ 6
@
8 years ago
Replying to swissspidy:
Should the function be called
wp_get_dependencies_unique_hosts()
as suggested in 50:ticket:34292?
I'm am not sure.
I think of the get_
and the_
prefixes as being somehow different but I can not explain how so succinctly. The prefixes put me in mind of WP objects, usually posts, whereas wp_[get_]dependencies_unique_hosts
works at a higher level within WordPress itself.
As I say, I am not sure.
Isn't there already a fix for that in another ticket? Can't find it right now though.