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

Unified Diff: mobile/android/base/java/org/adblockplus/browser/SubscriptionPreferenceCategory.java

Issue 29863604: Issue 6865 - Update ABP dependency to version 3.2 (Closed)
Patch Set: Adjusting build script and removing extension first run page Created Jan. 8, 2019, 2:54 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: mobile/android/base/java/org/adblockplus/browser/SubscriptionPreferenceCategory.java
===================================================================
--- a/mobile/android/base/java/org/adblockplus/browser/SubscriptionPreferenceCategory.java
+++ b/mobile/android/base/java/org/adblockplus/browser/SubscriptionPreferenceCategory.java
@@ -48,21 +48,21 @@
super(context, attrs);
}
public SubscriptionPreferenceCategory(Context context, AttributeSet attrs, int defStyleAttr)
{
super(context, attrs, defStyleAttr);
}
- private static synchronized void initSubscriptions()
+ private static synchronized void initSubscriptions(Context context)
jens 2019/01/10 14:10:59 Context could be final.
diegocarloslima 2019/01/16 13:44:22 Acknowledged.
{
if (subscriptionContainer == null)
{
- subscriptionContainer = SubscriptionContainer.create();
+ subscriptionContainer = SubscriptionContainer.create(context);
}
else
{
subscriptionContainer.refresh();
}
}
@Override
@@ -72,22 +72,22 @@
this.setEnabled(false);
this.setShouldDisableView(true);
super.onAttachedToActivity();
showProgressDialog();
- AddOnBridge.postToHandler(new Runnable()
+ ExtensionBridge.postToHandler(new Runnable()
{
@Override
public void run()
{
- initSubscriptions();
+ initSubscriptions(getContext());
subscriptionContainer.addSubscriptionListener(
SubscriptionPreferenceCategory.this.subscriptionChangeListener);
ThreadUtils.postToUiThread(new Runnable()
{
@Override
public void run()
{
@@ -201,9 +201,8 @@
{
@Override
public void onSubscriptionUpdated()
{
SubscriptionPreferenceCategory.this.refreshEntries();
}
}
}
-

Powered by Google App Engine
This is Rietveld