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

Unified Diff: libadblockplus-android/src/org/adblockplus/libadblockplus/android/AndroidWebRequestResourceWrapper.java

Issue 29411633: Issue 5136 - Add test for AndroidWebRequestResourceWrapper (Closed)
Patch Set: Created April 13, 2017, 1:40 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: libadblockplus-android/src/org/adblockplus/libadblockplus/android/AndroidWebRequestResourceWrapper.java
diff --git a/libadblockplus-android/src/org/adblockplus/libadblockplus/android/AndroidWebRequestResourceWrapper.java b/libadblockplus-android/src/org/adblockplus/libadblockplus/android/AndroidWebRequestResourceWrapper.java
index 4af10c8cfbabeee4ebfad686df38e9c8ae7dab60..4ec1d4a5bf415cfcb1f9f29a4f1f47db9275d8f5 100644
--- a/libadblockplus-android/src/org/adblockplus/libadblockplus/android/AndroidWebRequestResourceWrapper.java
+++ b/libadblockplus-android/src/org/adblockplus/libadblockplus/android/AndroidWebRequestResourceWrapper.java
@@ -112,7 +112,7 @@ public class AndroidWebRequestResourceWrapper extends WebRequest
public ServerResponse httpGET(String url, List<HeaderEntry> headers)
{
// since parameters may vary we need to ignore them
- String urlWithoutParams = url.substring(0, url.indexOf("?"));
+ String urlWithoutParams = Utils.getUrlWithoutParams(url);
anton 2017/04/13 13:46:50 actually extracting url without params isn't a dut
Integer resourceId = urlToResourceIdMap.get(urlWithoutParams);
if (resourceId != null)

Powered by Google App Engine
This is Rietveld