Opened on 01/13/2015 at 02:52:42 PM
Closed on 12/01/2015 at 11:17:55 AM
Last modified on 12/19/2016 at 12:34:10 AM
#1793 closed change (fixed)
Check whether the frame is whitelisted before injecting CSS
Reported by: | sergz | Assignee: | sergz |
---|---|---|---|
Priority: | P3 | Milestone: | Adblock-Plus-for-Internet-Explorer-1.6 |
Module: | Adblock-Plus-for-Internet-Explorer | Keywords: | |
Cc: | oleksandr, eric@… | Blocked By: | #1794, #1795, #3412 |
Blocking: | #119 | Platform: | Internet Explorer |
Ready: | yes | Confidential: | no |
Tester: | Rraceanu | Verified working: | yes |
Review URL(s): |
Description
Background
Before injecting CSS we should test whether we need to inject it or not.
What to change
Currently FilterEngine::Matches already accepts frame hierarchy but the implementation does not look completely correct. So, to be consistent with other platforms let's implement the handling of frame hierarchy manually as it's done for Chrome https://hg.adblockplus.org/adblockpluschrome/file/9b341ec66249/lib/whitelisting.js#l36. I guess later we can move it into libadblockplus.
Attachments (0)
Change History (10)
comment:1 Changed on 01/13/2015 at 02:56:44 PM by sergz
- Blocked By 1794 added
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:2 Changed on 01/13/2015 at 03:24:45 PM by sergz
- Blocked By 1795 added
comment:3 Changed on 01/13/2015 at 03:27:15 PM by sergz
- Blocking 119 added
comment:4 Changed on 03/16/2015 at 12:04:07 PM by oleksandr
- Platform changed from Unknown to Internet Explorer
- Priority changed from Unknown to P3
- Ready set
comment:5 Changed on 11/16/2015 at 10:25:14 AM by sergz
- Summary changed from Check whether the frame is whitellisted before injecting CSS to Check whether the frame is whitelisted before injecting CSS
- Tester set to Unknown
- Verified working unset
comment:6 Changed on 12/01/2015 at 11:17:55 AM by sergz
- Resolution set to fixed
- Status changed from reviewing to closed
comment:7 Changed on 12/15/2015 at 01:27:04 AM by oleksandr
- Blocked By 3412 added
comment:8 Changed on 10/18/2016 at 01:56:48 PM by oleksandr
- Milestone set to Adblock-Plus-for-Internet-Explorer-Next
comment:9 Changed on 11/08/2016 at 12:59:41 AM by rraceanu
- Tester changed from Unknown to Rraceanu
Note: See
TracTickets for help on using
tickets.