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

Unified Diff: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptions.java

Issue 29505565: Issue 5237 - Correct filter lists not applied on other language installation (Closed)
Patch Set: Issue 5237 - Correct filter lists not applied on other language installation Created Aug. 4, 2017, 2:30 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: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptions.java
===================================================================
--- a/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptions.java
+++ b/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptions.java
@@ -88,16 +88,21 @@ final class DefaultSubscriptions
return this.urlMap.get(url);
}
public DefaultSubscriptionInfo getForUrl(final URL url)
{
return url != null ? this.getForUrl(url.toString()) : null;
}
+ List<DefaultSubscriptionInfo> getAdsSubscriptions()
anton 2017/08/08 05:43:41 why not public (package now)?
jens 2017/08/08 13:53:05 I thought it was to keep this method package-priva
anton 2017/08/11 11:25:07 That looks strange. Please consider making it 'pri
jens 2017/08/11 13:27:23 Acknowledged.
+ {
+ return adsSubscriptions;
+ }
+
private void listSubscriptions(final List<DefaultSubscriptionInfo> output)
{
for (final DefaultSubscriptionInfo s : this.subscriptions)
{
this.listSubscriptions(s, output);
}
}

Powered by Google App Engine
This is Rietveld