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

Side by Side Diff: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscriptions.java

Issue 29445587: Issue 5223 - App is freezing sometimes (Closed)
Patch Set: Fixed formatting and set dialog = null Created June 26, 2017, 2:04 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2016 Eyeo GmbH 3 * Copyright (C) 2006-2016 Eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 { 112 {
113 if (enabled != sub.isEnabled()) 113 if (enabled != sub.isEnabled())
114 { 114 {
115 sub.setEnabled(enabled); 115 sub.setEnabled(enabled);
116 sub.serializeMetaData(this.getMetaFile(sub)); 116 sub.serializeMetaData(this.getMetaFile(sub));
117 if (enabled) 117 if (enabled)
118 { 118 {
119 this.engine.enqueueDownload(sub, true); 119 this.engine.enqueueDownload(sub, true);
120 } 120 }
121 121
122 this.engine.subscriptionStateChanged();
122 this.engine.requestUpdateBroadcast(); 123 this.engine.requestUpdateBroadcast();
123 return true; 124 return true;
124 } 125 }
125 } 126 }
126 return false; 127 return false;
127 } 128 }
128 129
129 File getFiltersFile(final Subscription sub) 130 File getFiltersFile(final Subscription sub)
130 { 131 {
131 final File filtersFile; 132 final File filtersFile;
(...skipping 154 matching lines...) Expand 10 before | Expand all | Expand 10 after
286 if (sub != null) 287 if (sub != null)
287 { 288 {
288 if (sub.updateSubscription(responseCode, text, httpHeaders, this.getMetaFi le(sub), 289 if (sub.updateSubscription(responseCode, text, httpHeaders, this.getMetaFi le(sub),
289 this.getFiltersFile(sub))) 290 this.getFiltersFile(sub)))
290 { 291 {
291 this.engine.requestUpdateBroadcast(); 292 this.engine.requestUpdateBroadcast();
292 } 293 }
293 } 294 }
294 } 295 }
295 } 296 }
OLDNEW

Powered by Google App Engine
This is Rietveld