Opened on 05/13/2016 at 09:44:54 AM
Closed on 05/17/2016 at 08:45:49 AM
Last modified on 09/07/2016 at 11:56:04 PM
#4041 closed defect (fixed)
Missing engine startup synchronization causing 'blocker not available' message
Reported by: | rjeschke | Assignee: | rjeschke |
---|---|---|---|
Priority: | P1 | Milestone: | Adblock-Plus-for-Samsung-Browser-1.0.3 |
Module: | Adblock-Plus-for-Samsung-Browser | Keywords: | |
Cc: | mario | Blocked By: | |
Blocking: | Platform: | Samsung Browser | |
Ready: | yes | Confidential: | no |
Tester: | Scheer | Verified working: | yes |
Review URL(s): |
Description
The content blocker's underlying engine is initialized asynchronously. Now, when the content blocker needs to be restarted, because SBrowser is asking for filters, then it is possibly that engine initialization isn't finished when SBrowser asks for the filters. This results in an error message thrown by us.
Currently the engine-initialized callback is executed on the UI thread, so it is not possible to wait for initialization in the content provider callback.
We need to implement two changes to fix this issue:
- allow the engine-initialized callback to be executed on the UI thread and a general purpose looper (possibly just a boolean for selecting which one to use)
- wait for engine initialization in the content provider callback
Attachments (0)
Change History (7)
comment:1 Changed on 05/13/2016 at 10:43:07 AM by rjeschke
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:2 Changed on 05/17/2016 at 08:45:02 AM by abpbot
comment:3 Changed on 05/17/2016 at 08:45:49 AM by rjeschke
- Resolution set to fixed
- Status changed from reviewing to closed
comment:4 Changed on 05/17/2016 at 09:30:28 AM by scheer
- Tester changed from Unknown to Scheer
- Verified working set
- The error message below no longer occurs:
ContentBlockerContentProvider: Engine not initialized SBrowserProxyHandler: getMDMProxy() : MDM proxy[] System.err: java.io.FileNotFoundException: Engine not yet initialized
- The error message in Sbrowser also no longer occurs 'blocker of unwanted content not available. Please contact the developer for details.'
Adblockplussbrowser-1.0.3-20160513123700-signed
Samsung Galaxy S5, S6
Android 5.0.1
Android 6.0
comment:5 Changed on 05/17/2016 at 10:28:54 AM by mario
- Milestone set to Adblock-Plus-for-Samsung-Browser-1.0.3
comment:6 Changed on 09/01/2016 at 04:04:29 PM by rhana@getadblock.com
Hi @rjeschke! We have a user who is still receiving this error with ABP for Samsung Browser v1.0.3. She initially reported this to us because she was receiving the error with AdBlock for Samsung Internet (both the original version and our last update, 1.2). Yesterday we had her install ABP 1.0.3 from Google Play. The error is still occurring. Is there something more she needs to do?
comment:7 Changed on 09/07/2016 at 11:56:04 PM by scheer
- Cc mario added
A commit referencing this issue has landed:
https://hg.adblockplus.org/adblockplussbrowser/rev/7f85855c3d8f