fix: remove all attributes can cause some errors when using livewire

Author: sebastienfontaine

src/content.js

@@ -162,10 +162,8 @@ export function processIcon(newIcon) {
 		// ⚠️ icon.remove() causes issues with some websites
 		// https://github.com/furybee/chrome-tab-modifier/issues/354
 		// icon.remove();
-		// Instead, remove all attributes
-		icon.getAttributeNames().forEach((attr) => {
-			icon.removeAttribute(attr);
-		});
+		// Instead, we'll just change the rel attribute
+		icon.setAttribute('rel', 'old-icon');
 	});
 
 	const iconUrl = /^(https?|data):/.test(newIcon)

src/content.test.js

@@ -248,7 +248,9 @@ describe('Content', () => {
 			processIcon('new-icon.png');
 			const oldIcon = document.querySelector('link[href="old-icon.png"]');
 			const newIcon = document.querySelector('link[href*="new-icon.png"]');
-			expect(oldIcon).toBeFalsy();
+
+			expect(oldIcon).toBeTruthy();
+			expect(oldIcon.rel).toBe('old-icon');
 			expect(newIcon).toBeTruthy();
 		});
 	});