work toward resolving #853

src/js/cookies.js

@@ -475,7 +475,7 @@ var canRemoveCookie = function(cookieKey, srcHostnames) {
     srcHostname = cookieHostname;
     var pos;
     for (;;) {
-        if ( srcHostnames.hasOwnProperty(srcHostname) ) {
+        if ( srcHostnames.has(srcHostname) ) {
             if ( µm.mustAllow(srcHostname, cookieHostname, 'cookie') ) {
                 return false;
             }

src/js/matrix.js

@@ -554,29 +554,21 @@ Matrix.prototype.evaluateSwitchZ = function(switchName, srcHostname) {
 
 /******************************************************************************/
 
-// TODO: In all likelyhood, will have to optmize here, i.e. keeping an
-// up-to-date collection of src hostnames with reference count etc.
+Matrix.prototype.extractAllSourceHostnames = (function() {
+    var cachedResult = new Set();
+    var readTime = 0;
 
-Matrix.prototype.extractAllSourceHostnames = function() {
-    var srcHostnames = {};
+    return function() {
+        if ( readTime !== this.modifiedTime ) {
+            cachedResult.clear();
             for ( var rule of this.rules.keys() ) {
-        srcHostnames[rule.slice(0, rule.indexOf(' '))] = true;
+                cachedResult.add(rule.slice(0, rule.indexOf(' ')));
             }
-    return srcHostnames;
-};
-
-/******************************************************************************/
-
-// TODO: In all likelyhood, will have to optmize here, i.e. keeping an
-// up-to-date collection of src hostnames with reference count etc.
-
-Matrix.prototype.extractAllDestinationHostnames = function() {
-    var desHostnames = {};
-    for ( var rule of this.rules.keys() ) {
-        desHostnames[this.desHostnameFromRule(rule)] = true;
+            readTime = this.modifiedTime;
         }
-    return desHostnames;
+        return cachedResult;
     };
+})();
 
 /******************************************************************************/