Unified Diff: libadblockplus-android/src/org/adblockplus/android/AndroidWebRequest.java
Issue 29351744:
Issue 4399 - Add WebView inheritor with ad blocking (Closed)
Patch Set: stopping for fast redirection
Created Sept. 27, 2016, 12:27 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« libadblockplus-android/src/org/adblockplus/android/AdblockWebView.java ('k')
|
libadblockplus-android/src/org/adblockplus/android/Utils.java » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: libadblockplus-android/src/org/adblockplus/android/AndroidWebRequest.java |
| diff --git a/libadblockplus-android/src/org/adblockplus/android/AndroidWebRequest.java b/libadblockplus-android/src/org/adblockplus/android/AndroidWebRequest.java |
| index 6796cd076d45eb8da866a989f02d6362df3756aa..d7d78039a2d804d9955917467628bdff6ed0bc6a 100644 |
| --- a/libadblockplus-android/src/org/adblockplus/android/AndroidWebRequest.java |
| +++ b/libadblockplus-android/src/org/adblockplus/android/AndroidWebRequest.java |
| @@ -92,35 +92,21 @@ public class AndroidWebRequest extends WebRequest |
| final BufferedReader reader = new BufferedReader(new InputStreamReader(connection.getInputStream(), "UTF-8")); |
| final StringBuilder sb = new StringBuilder(); |
| - if (!this.elemhideEnabled && isListedSubscriptionUrl(url)) |
| + String line; |
| + while ((line = reader.readLine()) != null) |
| { |
| - Log.d(TAG, "Removing element hiding rules from: '" + url + "'"); |
| - |
| - String line; |
| - while ((line = reader.readLine()) != null) |
| - { |
| - // We're only appending non-element-hiding filters here. |
| - // |
| - // See: |
| - // https://issues.adblockplus.org/ticket/303 |
| - // |
| - // Follow-up issue for removing this hack: |
| - // https://issues.adblockplus.org/ticket/1541 |
| - // |
| - if (line.indexOf('#') == -1) |
| - { |
| - sb.append(line); |
| - sb.append('\n'); |
| - } |
| - } |
| - } |
| - else |
| - { |
| - int character; |
| - |
| - while ((character = reader.read()) != -1) |
| + // We're only appending non-element-hiding filters here. |
| + // |
| + // See: |
| + // https://issues.adblockplus.org/ticket/303 |
| + // |
| + // Follow-up issue for removing this hack: |
| + // https://issues.adblockplus.org/ticket/1541 |
| + // |
| + if (this.elemhideEnabled || !isListedSubscriptionUrl(url) || line.indexOf('#') == -1) |
| { |
| - sb.append((char) character); |
| + sb.append(line); |
| + sb.append('\n'); |
| } |
| } |
| @@ -133,10 +119,12 @@ public class AndroidWebRequest extends WebRequest |
| { |
| response.setStatus(NsStatus.ERROR_FAILURE); |
| } |
| + Log.d(TAG, "Downloading finished"); |
| return response; |
| } |
| catch (final Throwable t) |
| { |
| + Log.e(TAG, "WebRequest failed", t); |
| throw new AdblockPlusException("WebRequest failed", t); |
| } |
| } |
« no previous file with comments
|
« libadblockplus-android/src/org/adblockplus/android/AdblockWebView.java ('k')
|
libadblockplus-android/src/org/adblockplus/android/Utils.java » ('j')
|
no next file with comments »
