commit acb91eb8e9d0f08a7ccce72062a7eb2cf958aa51
author Jay Shrauner <shrauner@google.com> Fri Aug 08 16:04:53 2014 -0700
committer Jay Shrauner <shrauner@google.com> Wed Aug 13 12:30:36 2014 -0700
tree 79acc868cbccf5011dff31e356394fd80c2fcc1c
parent 6fb37c87836b5245046bd3b14320823ab839a10c

Remove precondition checks

Remove precondition checks when we can work around them.

Bug:16620302
Change-Id: Ideb70a35c08f2275361eebec2850f454cf0bec55

src/com/android/telecomm/CallsManager.java

diff --git a/src/com/android/telecomm/CallsManager.java b/src/com/android/telecomm/CallsManager.java
index 5f01cb8..74b6e44 100644
--- a/src/com/android/telecomm/CallsManager.java
+++ b/src/com/android/telecomm/CallsManager.java
@@ -24,7 +24,6 @@
 import android.telecomm.PhoneAccountHandle;
 import android.telephony.DisconnectCause;
 
-import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableCollection;
 import com.google.common.collect.ImmutableList;
 
@@ -36,7 +35,7 @@
 /**
  * Singleton.
  *
- * NOTE(gilad): by design most APIs are package private, use the relevant adapter/s to allow
+ * NOTE: by design most APIs are package private, use the relevant adapter/s to allow
  * access from other packages specifically refraining from passing the CallsManager instance
  * beyond the com.android.telecomm package boundary.
  */
@@ -638,10 +637,11 @@
      * @param service The connection service that disconnected.
      */
     void handleConnectionServiceDeath(ConnectionServiceWrapper service) {
-        Preconditions.checkNotNull(service);
-        for (Call call : ImmutableList.copyOf(mCalls)) {
-            if (call.getConnectionService() == service) {
-                markCallAsDisconnected(call, DisconnectCause.ERROR_UNSPECIFIED, null);
+        if (service != null) {
+            for (Call call : ImmutableList.copyOf(mCalls)) {
+                if (call.getConnectionService() == service) {
+                    markCallAsDisconnected(call, DisconnectCause.ERROR_UNSPECIFIED, null);
+                }
             }
         }
     }
@@ -770,7 +770,9 @@
      * @param newState The new state of the call.
      */
     private void setCallState(Call call, int newState) {
-        Preconditions.checkNotNull(newState);
+        if (call == null) {
+            return;
+        }
         int oldState = call.getState();
         Log.i(this, "setCallState %s -> %s, call: %s", oldState, newState, call);
         if (newState != oldState) {