this fixes #156

js/abp-hide-filters.js

@@ -258,19 +258,19 @@ SelectorCacheEntry.prototype.addCosmetic = function(selectors) {
     }
 };
 
-SelectorCacheEntry.prototype.addNet = function(selector) {
-    if ( typeof selector !== 'string' || selector === '' ) {
-        return;
+SelectorCacheEntry.prototype.addNet = function(selectors) {
+    if ( typeof selectors === 'string' ) {
+        this.addNetOne(selectors, Date.now());
+    } else {
+        this.addNetMany(selectors, Date.now());
     }
     // Net request-derived selectors: I limit the number of cached selectors,
     // as I expect cases where the blocked net-requests are never the
     // exact same URL.
-    var dict = this.net;
-    if ( dict[selector] !== undefined ) {
-        dict[selector] = Date.now();
+    if ( this.netCount < this.netHighWaterMark ) {
         return;
     }
-    if ( this.netCount >= this.netHighWaterMark ) {
+    var dict = this.net;
     var keys = Object.keys(dict).sort(function(a, b) {
         return dict[b] - dict[a];
     }).slice(this.netLowWaterMark);
@@ -278,9 +278,27 @@ SelectorCacheEntry.prototype.addNet = function(selector) {
     while ( i-- ) {
         delete dict[keys[i]];
     }
-    }
-    dict[selector] = Date.now();
+};
+
+SelectorCacheEntry.prototype.addNetOne = function(selector, now) {
+    var dict = this.net;
+    if ( dict[selector] === undefined ) {
         this.netCount += 1;
+    }
+    dict[selector] = now;
+};
+
+SelectorCacheEntry.prototype.addNetMany = function(selectors, now) {
+    var dict = this.net;
+    var i = selectors.length || 0;
+    var selector;
+    while ( i-- ) {
+        selector = selectors[i];
+        if ( dict[selector] === undefined ) {
+            this.netCount += 1;
+        }
+        dict[selector] = now;
+    }
 };
 
 SelectorCacheEntry.prototype.add = function(selectors, type) {

js/contentscript-end.js

@@ -504,6 +504,53 @@ var uBlockMessaging = (function(name){
 (function() {
     var messaging = uBlockMessaging;
 
+    var blockableElements = {
+        'embed': 'src',
+        'iframe': 'src',
+        'img': 'src',
+        'object': 'data'
+    };
+
+    // First pass
+    messaging.ask({ what: 'blockedRequests' }, function(details) {
+        var elems = document.querySelectorAll('embed,iframe,img,object');
+        var blockedRequests = details.blockedRequests;
+        var collapse = details.collapse;
+        var i = elems.length;
+        var elem, tagName, prop, src;
+        var selectors = [];
+        while ( i-- ) {
+            elem = elems[i];
+            tagName = elem.tagName.toLowerCase();
+            prop = blockableElements[tagName];
+            if ( prop === undefined ) {
+                continue;
+            }
+            src = elem[prop];
+            if ( typeof src !== 'string' || src === '' ) {
+                continue;
+            }
+            if ( blockedRequests[src] === undefined ) {
+                continue;
+            }
+            // If `!important` is not there, going back using history will
+            // likely cause the hidden element to re-appear.
+            elem.style.visibility = 'hidden !important';
+            if ( collapse ) {
+                elem.parentNode.removeChild(elem);
+            }
+            selectors.push(tagName + '[' + prop + '="' + src + '"]');
+        }
+        if ( selectors.length !== 0 ) {
+            messaging.tell({
+                what: 'injectedSelectors',
+                type: 'net',
+                hostname: window.location.hostname,
+                selectors: selectors
+            });
+        }
+    });
+
     // Listeners to mop up whatever is otherwise missed:
     // - Future requests not blocked yet
     // - Elements dynamically added to the page
@@ -542,7 +589,7 @@ var uBlockMessaging = (function(name){
             // If `!important` is not there, going back using history will
             // likely cause the hidden element to re-appear.
             target.style.visibility = 'hidden !important';
-            if ( details.collapse ) {
+            if ( details.collapse && target.parentNode ) {
                 target.parentNode.removeChild(target);
             }
             messaging.tell({
@@ -556,12 +603,12 @@ var uBlockMessaging = (function(name){
     };
 
     var onResourceLoaded = function(ev) {
-        //console.debug('Loaded %s[src="%s"]', target.tagName, target.src);
+        //console.debug('Loaded %s[src="%s"]', ev.target.tagName, ev.target.src);
         onResource(ev.target, loadedElements);
     };
 
     var onResourceFailed = function(ev) {
-        //console.debug('Failed to load %o[src="%s"]', target, target.src);
+        //console.debug('Failed to load %o[src="%s"]', eev.target.tagName, ev.target.src);
         onResource(ev.target, failedElements);
     };
 

js/messaging-handlers.js

@@ -170,6 +170,13 @@ var onMessage = function(request, sender, callback) {
             µb.abpHideFilters.addToSelectorCache(request);
             break;
 
+        case 'blockedRequests':
+            response = {
+                collapse: µb.userSettings.collapseBlocked,
+                blockedRequests: pageStore ? pageStore.blockedRequests : {}
+            };
+            break;
+
         // Check a single request
         case 'blockedRequest':
             response = {