Issue 6389 - ABP for Samsung Internet Crashes when closed on specific devices

adblockplussbrowser/src/main/java/org/adblockplus/sbrowser/contentblocker/engine/EngineService.java

Index: adblockplussbrowser/src/main/java/org/adblockplus/sbrowser/contentblocker/engine/EngineService.java
===================================================================
--- a/adblockplussbrowser/src/main/java/org/adblockplus/sbrowser/contentblocker/engine/EngineService.java
+++ b/adblockplussbrowser/src/main/java/org/adblockplus/sbrowser/contentblocker/engine/EngineService.java
@@ -33,17 +33,17 @@ public final class EngineService extends
   private volatile boolean isInitialized = false;
   private Throwable failureCause = null;
   private static final LinkedBlockingQueue<EngineCreatedCallbackWrapper> ON_CREATED_CALLBACKS =
       new LinkedBlockingQueue<>();
 
   @Override
   public int onStartCommand(Intent intent, int flags, int startId)
   {
-    return Service.START_STICKY;
+    return Service.START_NOT_STICKY;

[anton, 2018/02/21 08:05:35]

it's not clear why it happens only on Google Pixel 1.
NOT_STICKY requires the service be started with `startService()`  each time and
i can't see related change (already working this way?).

[jens, 2018/02/21 09:25:55]

START_NOT_STICK does not trigger the service again until the next explicit call
to Context.startService(Intent). See also my comment in the trac ticket.

   }
 
   /**
    * The callback gets executed on the UI thread.
    *
    * @param context
    * @param callback
    */