fix(common): use DOCUMENT token to query for preconnect links (#47353)

`PreconnectLinkChecker` checks to see if preconnect links have been added to the `<head>` element but uses `document` directly which does not exist when rendering in Angular Universal. This PR switches the `PreconnectLinkChecker` to use the `DOCUMENT` token instead so that the query works when SSR'ing PR Close #47353
angular · Sep 6, 2022 · 1875ce5 · 1875ce5
1 parent c0d7ac9
commit 1875ce5
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/common/src/directives/ng_optimized_image/preconnect_link_checker.ts b/packages/common/src/directives/ng_optimized_image/preconnect_link_checker.ts
@@ -49,6 +49,8 @@ export const PRECONNECT_CHECK_BLOCKLIST =
  */
 @Injectable({providedIn: 'root'})
 export class PreconnectLinkChecker {
+  private document = inject(DOCUMENT);
+
   /**
    * Set of <link rel="preconnect"> tags found on this page.
    * The `null` value indicates that there was no DOM query operation performed.
@@ -66,7 +68,7 @@ export class PreconnectLinkChecker {
 
   constructor() {
     assertDevMode('preconnect link checker');
-    const win = inject(DOCUMENT).defaultView;
+    const win = this.document.defaultView;
     if (typeof win !== 'undefined') {
       this.window = win;
     }
@@ -127,7 +129,7 @@ export class PreconnectLinkChecker {
   private queryPreconnectLinks(): Set<string> {
     const preconnectUrls = new Set<string>();
     const selector = 'link[rel=preconnect]';
-    const links: HTMLLinkElement[] = Array.from(document.querySelectorAll(selector));
+    const links: HTMLLinkElement[] = Array.from(this.document.querySelectorAll(selector));
     for (let link of links) {
       const url = getUrl(link.href, this.window!);
       preconnectUrls.add(url.origin);