From b213d103d34a5f2ebbdafef6a7510fce2cc1e337 Mon Sep 17 00:00:00 2001
From: Brad Fitzpatrick <bradfitz@android.com>
Date: Mon, 19 Apr 2010 11:58:52 -0700
Subject: Reduce lock contention, removing unnecessary synchronization.

The value returned is purely a function of what's passed in, all
following final members.

BUG=2606839

Change-Id: I506fb9b1f3be965576c2b3317a30ff1f990ccda3
---
 .../java/com/android/server/am/ActivityManagerService.java     | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

(limited to 'services')

diff --git a/services/java/com/android/server/am/ActivityManagerService.java b/services/java/com/android/server/am/ActivityManagerService.java
index f8f8a0d..7e095b5 100644
--- a/services/java/com/android/server/am/ActivityManagerService.java
+++ b/services/java/com/android/server/am/ActivityManagerService.java
@@ -6141,12 +6141,10 @@ public final class ActivityManagerService extends ActivityManagerNative implemen
         if (!(pendingResult instanceof PendingIntentRecord)) {
             return null;
         }
-        synchronized(this) {
-            try {
-                PendingIntentRecord res = (PendingIntentRecord)pendingResult;
-                return res.key.packageName;
-            } catch (ClassCastException e) {
-            }
+        try {
+            PendingIntentRecord res = (PendingIntentRecord)pendingResult;
+            return res.key.packageName;
+        } catch (ClassCastException e) {
         }
         return null;
     }
-- 
cgit v1.1