Issue 6774 - $document doesn't whitelist everything inside iframes in Edge

ext/background.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present 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 547 matching lines @@
 
           if (!frames)
             return null;
 
           let frame;
           // In Microsoft Edge (version 42.17134.1.0) we don't have frameId
           // so we fall back to iterating over the tab's frames
           // see https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/11716733
           if (rawSender.frameId != undefined)
             frame = frames.get(rawSender.frameId);
-          else
+          else if (rawSender.url)
           {
-            let rawSenderHref = rawSender.url ?

[Sebastian Noack, 2018/07/19 16:18:05]

Do we even want to enter this code path if we don't have a URL?

[geo, 2018/07/19 18:48:16]

No, you are right, there is no point to it. `frameInfo` will always have an url,
so there will never be a match.
Done. 

-              rawSender.url.replace(/#.*/, "") :
-              "";
+            let rawSenderHref = rawSender.url.replace(/#.*/, "");
 
             for (let [frameId, frameInfo] of frames)
             {
               let frameInfoHref = frameInfo.url.href.replace(/#.*/, "");
 
               // If we have two frames with the same URL
               // we are going to pick the first one we find
               // as we have no other way of distinguishing between them.
               if (frameInfoHref == rawSenderHref)
               {
                 frame = frameInfo;
                 this.id = frameId;
                 break;
               }
             }
           }
 
           if (frame)
             return frame.parent || null;
           return frames.get(0) || null;
         }
       };
     }
 
     return ext.onMessage._dispatch(
       message, sender, sendResponse
     ).includes(true);
   });
 }