Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: ext/background.js

Issue 29832561: Issue 6774 - $document doesn't whitelist everything inside iframes in Edge (Closed)
Patch Set: Created July 19, 2018, 6:39 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-present eyeo GmbH 3 * Copyright (C) 2006-present eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 541 matching lines...) Expand 10 before | Expand all | Expand 10 after
552 // In Edge requests from internal extension pages 552 // In Edge requests from internal extension pages
553 // (protocol ms-browser-extension://) do no have a sender URL. 553 // (protocol ms-browser-extension://) do no have a sender URL.
554 url: rawSender.url ? new URL(rawSender.url) : null, 554 url: rawSender.url ? new URL(rawSender.url) : null,
555 get parent() 555 get parent()
556 { 556 {
557 let frames = framesOfTabs.get(rawSender.tab.id); 557 let frames = framesOfTabs.get(rawSender.tab.id);
558 558
559 if (!frames) 559 if (!frames)
560 return null; 560 return null;
561 561
562 let frame = frames.get(rawSender.frameId); 562 let frame;
563 // In Microsoft Edge (version 42.17134.1.0) we don't have frameId
564 // so we fall back to iterating over the tab's frames
565 // see https://developer.microsoft.com/en-us/microsoft-edge/platform/i ssues/11716733
566 if (rawSender.frameId != undefined)
567 frame = frames.get(rawSender.frameId);
568 else if (rawSender.url)
569 {
570 let rawSenderHref = rawSender.url.replace(/#.*/, "");
571
572 for (let [frameId, frameInfo] of frames)
573 {
574 let frameInfoHref = frameInfo.url.href.replace(/#.*/, "");
575
576 // If we have two frames with the same URL
577 // we are going to pick the first one we find
578 // as we have no other way of distinguishing between them.
579 if (frameInfoHref == rawSenderHref)
580 {
581 frame = frameInfo;
582 this.id = frameId;
583 break;
584 }
585 }
586 }
587
563 if (frame) 588 if (frame)
564 return frame.parent || null; 589 return frame.parent || null;
565
566 return frames.get(0) || null; 590 return frames.get(0) || null;
567 } 591 }
568 }; 592 };
569 } 593 }
570 594
571 return ext.onMessage._dispatch( 595 return ext.onMessage._dispatch(
572 message, sender, sendResponse 596 message, sender, sendResponse
573 ).includes(true); 597 ).includes(true);
574 }); 598 });
575 } 599 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld