Issue 3258 - Blockable items: restore item flashing functionality

lib/child/requestNotifier.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 52 matching lines @@
   let {notifierID, requests, scrollToItem} = message.data;
   let notifier = notifiers.get(notifierID);
   if (notifier)
     notifier.flashNodes(requests, scrollToItem);
 }
 
 /**
  * Creates a notifier object for a particular window. After creation the window
  * will first be scanned for previously saved requests. Once that scan is
  * complete only new requests for this window will be reported.
- * @param {Window} wnd  window to attach the notifier to
- * @param {Integer} id  Parent notifier ID to be messaged
+ * @param {Window} window  window to attach the notifier to
+ * @param {Integer} notifierID  Parent notifier ID to be messaged
  */
 function RequestNotifier(window, notifierID)
 {
   this.window = window;
   this.id = notifierID;
   notifiers.set(this.id, this);
   this.nodes = new Map();
   this.startScan(window);
 }
 exports.RequestNotifier = RequestNotifier;
@@ skipping 60 matching lines @@
    * Starts the initial scan of the window (will recurse into frames).
    * @param {Window} wnd  the window to be scanned
    */
   startScan: function(wnd)
   {
     let doc = wnd.document;
     let walker = doc.createTreeWalker(doc, Ci.nsIDOMNodeFilter.SHOW_ELEMENT, null, false);
 
     let process = function()
     {
+      // Don't do anything if the notifier was shut down already.
       if (!this.window)
         return;
 
       let node = walker.currentNode;
       let data = nodeData.get(node);
       if (typeof data != "undefined")
         for (let k in data)
           this.notifyListener(node, data[k]);
 
       if (walker.nextNode())
@@ skipping 15 matching lines @@
 
     // Process each node in a separate event to allow other events to process
     this.eventsPosted++;
     Utils.runAsync(process);
   },
 
   /**
    * Makes the nodes associated with the given requests blink.
    * @param {number[]} requests  list of request IDs that were previously
    *                             reported by this notifier.
-   * @param {Boolean} scrollToItem  if true, scroll to first node
+   * @param {boolean} scrollToItem  if true, scroll to first node
    */
   flashNodes: function(requests, scrollToItem)
   {
     this.stopFlashing();
 
     let nodes = [];
     for (let id of requests)
     {
+      if (!this.nodes.has(id))
+        continue;
+
       let node = this.nodes.get(id);
-      if (typeof node != "undefined" && node.nodeType == Ci.nsIDOMNode.ELEMENT_NODE)
+      if (Cu.isDeadWrapper(node))
+        this.nodes.delete(node);
+      else if (node.nodeType == Ci.nsIDOMNode.ELEMENT_NODE)
         nodes.push(node);
     }
     if (nodes.length)
       this.flasher = new Flasher(nodes, scrollToItem);
   },
 
   /**
    * Stops flashing nodes after a previous flashNodes() call.
    */
   stopFlashing: function()
@@ skipping 61 matching lines @@
       stats.filters[filter.text]++;
     else
       stats.filters[filter.text] = 1;
   }
 
   // Notify listeners
   for (let notifier of notifiers.values())
     if (!notifier.window || notifier.window == topWnd)
       notifier.notifyListener(node, entry);
 }