make processLinks switch to imperative style to be cleaner and consis…

…tent

web/autolinker.js

@@ -71,20 +71,21 @@ class Autolinker {
   static processLinks(pdfPageView, textContent) {
     const [text, diffs] = normalize(textContent.join(""));
     const matches = text.matchAll(Autolinker.#urlRegex);
-    return Array.from(matches, match => {
+    const links = [];
+    for (const match of matches) {
       const url = createValidAbsoluteUrl(match[0]);
       if (url) {
         const [index, length] = getOriginalIndex(
           diffs,
           match.index,
           match[0].length
         );
-        return this.#addLinkAnnotations(url.href, index, length, pdfPageView);
+        links.push(
+          ...this.#addLinkAnnotations(url.href, index, length, pdfPageView)
+        );
       }
-      return url;
-    })
-      .filter(annotation => annotation !== null)
-      .flat();
+    }
+    return links;
   }
 }