Unified Diff: src/org/adblockplus/android/AdblockPlus.java
Issue 5763633376133120:
Don't inject any element hiding rules (Closed)
Patch Set:
Created Nov. 29, 2013, 1:10 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: src/org/adblockplus/android/AdblockPlus.java |
| =================================================================== |
| --- a/src/org/adblockplus/android/AdblockPlus.java |
| +++ b/src/org/adblockplus/android/AdblockPlus.java |
| @@ -292,10 +292,20 @@ |
| */ |
| public String[] getSelectorsForDomain(final String domain) |
| { |
| + /* We need to ignore element hiding rules here to work around two bugs: |
| + * 1. CSS is being injected even when there's an exception rule with $elemhide |
| + * 2. The injected CSS causes blank pages in Chrome for Android |
| + * |
| + * Starting with 1.1.2, we ignored element hiding rules after download anyway, to keep the |
| + * memory usage down. Doing this with libadblockplus is trickier, but would be the clean |
| + * solution. */ |
| + return null; |
| +/* |
| if (!filteringEnabled) |
| return null; |
| return abpEngine.getSelectorsForDomain(domain); |
| +*/ |
| } |
| /** |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
