Better setup / teardown for JobService (#745)

* Better setup / teardown for JobService

* Move to lazy pattern

Author: natario1

Parse/src/main/java/com/parse/PushServiceApi26.java

@@ -25,6 +25,7 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.concurrent.Executor;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 
@@ -44,7 +45,8 @@ static boolean run(Context context, Intent intent) {
     // Execute in the next second.
     Bundle extra = new Bundle(1);
     extra.putParcelable(INTENT_KEY, intent);
-    int did = scheduler.schedule(new JobInfo.Builder(JOB_SERVICE_ID, new ComponentName(context, PushServiceApi26.class))
+    ComponentName component = new ComponentName(context, PushServiceApi26.class);
+    int did = scheduler.schedule(new JobInfo.Builder(JOB_SERVICE_ID, component)
         .setMinimumLatency(1L)
         .setOverrideDeadline(1000L)
         .setRequiresCharging(false)
@@ -58,19 +60,13 @@ static boolean run(Context context, Intent intent) {
   // We delegate the intent to a PushHandler running in a streamlined executor.
   private ExecutorService executor;
   private PushHandler handler;
+  private int jobsCount;
 
   // Our manifest file is OK.
   static boolean isSupported() {
     return true;
   }
 
-  @Override
-  public void onCreate() {
-    super.onCreate();
-    executor = Executors.newSingleThreadExecutor();
-    handler = PushServiceUtils.createPushHandler();
-  }
-
   @Override
   public boolean onStartJob(final JobParameters jobParameters) {
     if (ParsePlugins.Android.get() == null) {
@@ -85,13 +81,18 @@ public boolean onStartJob(final JobParameters jobParameters) {
 
     final Bundle params = jobParameters.getTransientExtras();
     final Intent intent = params.getParcelable(INTENT_KEY);
-    executor.execute(new Runnable() {
+    jobsCount++;
+    getExecutor().execute(new Runnable() {
       @Override
       public void run() {
         try {
-          handler.handlePush(intent);
+          getHandler().handlePush(intent);
         } finally {
           jobFinished(jobParameters, false);
+          jobsCount--;
+          if (jobsCount == 0) {
+            tearDown();
+          }
         }
       }
     });
@@ -104,13 +105,19 @@ public boolean onStopJob(JobParameters jobParameters) {
     return true;
   }
 
-  @Override
-  public void onDestroy() {
-    if (executor != null) {
-      executor.shutdown();
-      executor = null;
-      handler = null;
-    }
-    super.onDestroy();
+  private Executor getExecutor() {
+    if (executor == null) executor = Executors.newSingleThreadExecutor();
+    return executor;
+  }
+
+  private PushHandler getHandler() {
+    if (handler == null) handler = PushServiceUtils.createPushHandler();
+    return handler;
+  }
+
+  private void tearDown() {
+    if (executor != null) executor.shutdown();
+    executor = null;
+    handler = null;
   }
 }