Simplify ElementOnce observer

core/static/bundled/core/components/include-index.ts

@@ -61,7 +61,25 @@ const startObserver = (observer: MutationObserver) => {
   });
 };
 
-// Refresh *-once components when changes happens
+// Refresh all ElementOnce components on the document based on their tag name
+// They should all be be extended from the `elementOnce` factory
+const refreshElement = (componentName: string, tagName: string) => {
+  for (const element of document.getElementsByTagName(componentName)) {
+    // Creating a base class would be really complicated, we can use any ElementOnce
+    // element instead to cast, it doesn't matter
+    const node = element as LinkOnce;
+
+    // We can't guess if an element is compatible before we get one
+    // We exit the function completely if it's not compatible
+    if (node.inheritedTagName.toUpperCase() !== tagName.toUpperCase()) {
+      return;
+    }
+
+    node.refresh();
+  }
+};
+
+// Refresh ElementOnce components when changes happens
 const observer = new MutationObserver((mutations: MutationRecord[]) => {
   observer.disconnect();
   for (const mutation of mutations) {
@@ -69,19 +87,6 @@ const observer = new MutationObserver((mutations: MutationRecord[]) => {
       if (node.nodeType !== node.ELEMENT_NODE) {
         continue;
       }
-      const refreshElement = (componentName: string, tagName: string) => {
-        for (const element of document.getElementsByTagName(componentName)) {
-          // We can't guess if an element is compatible before we get one
-          // We exit the function completely if it's not compatible
-          if (
-            (element as any).inheritedTagName.toUpperCase() !== tagName.toUpperCase()
-          ) {
-            return;
-          }
-
-          (element as any).refresh();
-        }
-      };
       for (const registered of registeredComponents) {
         refreshElement(registered, (node as HTMLElement).tagName);
       }