Fix wake lock leak am: cfc6b97c7d am: 546c1ffab6

am: 59f758a01d

* commit '59f758a01daf2f0bdd46aa33b2dcf6ae45a457c1':
  Fix wake lock leak

1 files changed, 32 insertions, 27 deletions

diff --git a/src/com/android/packageinstaller/wear/WearPackageInstallerService.java b/src/com/android/packageinstaller/wear/WearPackageInstallerService.java
index 6dc5aa70..229a3df6 100644
--- a/src/com/android/packageinstaller/wear/WearPackageInstallerService.java
+++ b/src/com/android/packageinstaller/wear/WearPackageInstallerService.java
@@ -586,31 +586,33 @@ public class WearPackageInstallerService extends Service {
         }
 
         public void packageInstalled(String packageName, int returnCode) {
-            // If installation failed, bail out and remove the ShowPermsStore entry
-            if (returnCode < 0) {
-                Log.e(TAG, "Package install failed " + mApplicationPackageName
-                        + ", returnCode " + returnCode);
-                WearPackageUtil.removeFromPermStore(mContext, mApplicationPackageName);
-                return;
-            }
+            try {
+                // If installation failed, bail out and remove the ShowPermsStore entry
+                if (returnCode < 0) {
+                    Log.e(TAG, "Package install failed " + mApplicationPackageName
+                            + ", returnCode " + returnCode);
+                    WearPackageUtil.removeFromPermStore(mContext, mApplicationPackageName);
+                    return;
+                }
 
-            Log.i(TAG, "Package " + packageName + " was installed.");
+                Log.i(TAG, "Package " + packageName + " was installed.");
 
-            // Delete tempFile from the file system.
-            File tempFile = WearPackageUtil.getTemporaryFile(mContext, packageName);
-            if (tempFile != null) {
-                tempFile.delete();
-            }
+                // Delete tempFile from the file system.
+                File tempFile = WearPackageUtil.getTemporaryFile(mContext, packageName);
+                if (tempFile != null) {
+                    tempFile.delete();
+                }
 
-            // Broadcast the "UPDATED" gmscore intent, normally sent by play store.
-            // TODO: Remove this broadcast if/when we get the play store to do this for us.
-            if (GMS_PACKAGE_NAME.equals(packageName)) {
-                Intent gmsInstalledIntent = new Intent(GMS_UPDATED_BROADCAST);
-                gmsInstalledIntent.setPackage(GMS_PACKAGE_NAME);
-                mContext.sendBroadcast(gmsInstalledIntent);
+                // Broadcast the "UPDATED" gmscore intent, normally sent by play store.
+                // TODO: Remove this broadcast if/when we get the play store to do this for us.
+                if (GMS_PACKAGE_NAME.equals(packageName)) {
+                    Intent gmsInstalledIntent = new Intent(GMS_UPDATED_BROADCAST);
+                    gmsInstalledIntent.setPackage(GMS_PACKAGE_NAME);
+                    mContext.sendBroadcast(gmsInstalledIntent);
+                }
+            } finally {
+                finishService(mWakeLock, mStartId);
             }
-
-            finishService(mWakeLock, mStartId);
         }
     }
 
@@ -624,13 +626,16 @@ public class WearPackageInstallerService extends Service {
         }
 
         public void packageDeleted(String packageName, int returnCode) {
-            if (returnCode >= 0) {
-                Log.i(TAG, "Package " + packageName + " was uninstalled.");
-            } else {
-                Log.e(TAG, "Package uninstall failed " + packageName + ", returnCode " +
-                        returnCode);
+            try {
+                if (returnCode >= 0) {
+                    Log.i(TAG, "Package " + packageName + " was uninstalled.");
+                } else {
+                    Log.e(TAG, "Package uninstall failed " + packageName + ", returnCode " +
+                            returnCode);
+                }
+            } finally {
+                finishService(mWakeLock, mStartId);
             }
-            finishService(mWakeLock, mStartId);
         }
     }
 }