commit 87ad1fce254bdb398c5a33753f0368a596fd81f0
author Chris Tate <ctate@android.com> Tue May 31 18:36:48 2016 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Tue May 31 18:36:49 2016 +0000
tree 9a7317f081e62870f9f754d2879c8fc5d1936cff
parent 8b655e0f57b071756e7d6a3547eeabe35885bcc9
parent 3eddecc3bc6e4f9f2d9b6a64d6d1f0addfb286a4

Merge "Build JobStatus objects outside the lock" into nyc-dev

services/core/java/com/android/server/job/JobServiceContext.java

diff --git a/services/core/java/com/android/server/job/JobServiceContext.java b/services/core/java/com/android/server/job/JobServiceContext.java
index 4fd1350..31528e5 100644
--- a/services/core/java/com/android/server/job/JobServiceContext.java
+++ b/services/core/java/com/android/server/job/JobServiceContext.java
@@ -223,10 +223,11 @@
      * stop executing.
      */
     JobStatus getRunningJob() {
+        final JobStatus job;
         synchronized (mLock) {
-            return mRunningJob == null ?
-                    null : new JobStatus(mRunningJob);
+            job = mRunningJob;
         }
+        return job == null ? null : new JobStatus(job);
     }
 
     /** Called externally when a job that was scheduled for execution should be cancelled. */