Workaround declarativeNetRequest element collapsing bug

There's a bug[1] with the declarativeNetRequest API that means that
after blocked iframes are collapsed (hidden) they sometimes can't be
uncollapsed (shown again) after they load successfully. That causes
Click to Load to get messed up for Chrome MV3 builds of the extension,
as the content is sometimes not shown to the user again after it has
been unblocked. The bug is triggered when the collapsed iframe is
removed from the DOM.

As a workaround, let's load a benign data URI in blocked iframes
before removing them from the DOM. That way they are uncollapsed, and
so if they are added back to the page they will be visible to the
user.

1 - https://crbug.com/1428971

Author: kzar

src/features/click-to-load.js

@@ -484,6 +484,16 @@ function replaceTrackingElement (widget, trackingElement, placeholderElement) {
     ]
     elementToReplace.style.setProperty('display', 'none', 'important')
 
+    // When iframes are blocked by the declarativeNetRequest API, they are
+    // collapsed (hidden) automatically. Unfortunately however, there's a bug
+    // that stops them from being uncollapsed (shown again) if they are removed
+    // from the DOM after they are collapsed. As a workaround, have the iframe
+    // load a benign data URI, so that it's uncollapsed, before removing it from
+    // the DOM. See https://crbug.com/1428971
+    const originalSrc = elementToReplace.src
+    elementToReplace.src =
+        'data:text/plain;charset=utf-8;base64,aHR0cHM6Ly9jcmJ1Zy5jb20vMTQyODk3MQ=='
+
     // Add the placeholder element to the page.
     elementToReplace.parentElement.insertBefore(
         placeholderElement, elementToReplace
@@ -503,6 +513,7 @@ function replaceTrackingElement (widget, trackingElement, placeholderElement) {
         // placeholder) can finally be removed from the DOM.
         elementToReplace.remove()
         elementToReplace.style.setProperty('display', ...originalDisplay)
+        elementToReplace.src = originalSrc
     })
 }