Misc phone app code cleanup.
David Brown [Fri, 26 Jun 2009 23:09:10 +0000 (16:09 -0700)]
- remove unused String "finalTitle" in NotificationMgr (accidentally
  included in one of the Moto changes)

- tweak bottom margin of one CallCard text field to avoid cutting off
  descenders

- remove the unused PhoneUtils getCompactName() methods

- remove unused CallCard showCall* methods

- remove useless CallCard reset() method

- When the radio technology changes, have PhoneApp call a new InCallScreen
  method that unregisters then re-registers for phone state events.
  (Even though we don't have any devices yet that can switch on the fly
  between GSM and CDMA, the code still ought to do this.)

- some minor debug logging cleanup

TESTED: ran basic telephony test cases.

res/layout/call_card_person_info.xml
src/com/android/phone/CallCard.java
src/com/android/phone/CallFeaturesSetting.java
src/com/android/phone/CallNotifier.java
src/com/android/phone/InCallScreen.java
src/com/android/phone/NotificationMgr.java
src/com/android/phone/PhoneApp.java
src/com/android/phone/PhoneUtils.java

index 981666e..69b0a8b 100644 (file)
@@ -57,7 +57,7 @@
               android:singleLine="true"
               android:layout_width="wrap_content"
               android:layout_height="wrap_content"
-              android:layout_marginBottom="4dip"
+              android:layout_marginBottom="6dip"
               android:paddingLeft="4dip"
               android:paddingRight="4dip"
               />
@@ -87,5 +87,4 @@
                   />
     </LinearLayout>
 
-
 </LinearLayout>
index 1ad3cf7..418cf40 100755 (executable)
@@ -137,14 +137,6 @@ public class CallCard extends FrameLayout
         mInCallScreen = inCallScreen;
     }
 
-    void reset() {
-        if (DBG) log("reset()...");
-
-        // default to show ACTIVE call style, with empty title and status text
-        showCallConnected();
-        setUpperTitle("");
-    }
-
     public void onTickForCallTimeElapsed(long timeElapsed) {
         // While a call is in progress, update the elapsed time shown
         // onscreen.
@@ -375,8 +367,6 @@ public class CallCard extends FrameLayout
 
         switch (state) {
             case ACTIVE:
-                showCallConnected();
-
                 if (bluetoothActive) {
                     callCardBackgroundResid =
                             landscapeMode ? R.drawable.incall_frame_bluetooth_tall_land
@@ -396,8 +386,6 @@ public class CallCard extends FrameLayout
                 break;
 
             case HOLDING:
-                showCallOnhold();
-
                 callCardBackgroundResid =
                         landscapeMode ? R.drawable.incall_frame_hold_tall_land
                         : R.drawable.incall_frame_hold_tall_port;
@@ -408,9 +396,6 @@ public class CallCard extends FrameLayout
                 break;
 
             case DISCONNECTED:
-                reset();
-                showCallEnded();
-
                 callCardBackgroundResid =
                         landscapeMode ? R.drawable.incall_frame_ended_tall_land
                         : R.drawable.incall_frame_ended_tall_port;
@@ -422,8 +407,6 @@ public class CallCard extends FrameLayout
 
             case DIALING:
             case ALERTING:
-                showCallConnecting();
-
                 if (bluetoothActive) {
                     callCardBackgroundResid =
                             landscapeMode ? R.drawable.incall_frame_bluetooth_tall_land
@@ -441,8 +424,6 @@ public class CallCard extends FrameLayout
 
             case INCOMING:
             case WAITING:
-                showCallIncoming();
-
                 if (bluetoothActive) {
                     callCardBackgroundResid =
                             landscapeMode ? R.drawable.incall_frame_bluetooth_tall_land
@@ -587,8 +568,6 @@ public class CallCard extends FrameLayout
         // Background images are also different if Bluetooth is active.
         final boolean bluetoothActive = mApplication.showBluetoothIndication();
 
-        showCallConnected();
-
         int callCardBackgroundResid = 0;
         if (bluetoothActive) {
             callCardBackgroundResid =
@@ -1033,30 +1012,6 @@ public class CallCard extends FrameLayout
         return getContext().getString(resID);
     }
 
-    private void showCallConnecting() {
-        if (DBG) log("showCallConnecting()...");
-        // TODO: remove if truly unused
-    }
-
-    private void showCallIncoming() {
-        if (DBG) log("showCallIncoming()...");
-        // TODO: remove if truly unused
-    }
-
-    private void showCallConnected() {
-        if (DBG) log("showCallConnected()...");
-        // TODO: remove if truly unused
-    }
-
-    private void showCallEnded() {
-        if (DBG) log("showCallEnded()...");
-        // TODO: remove if truly unused
-    }
-    private void showCallOnhold() {
-        if (DBG) log("showCallOnhold()...");
-        // TODO: remove if truly unused
-    }
-
     /**
      * Updates the name / photo / number / label fields on the CallCard
      * based on the specified CallerInfo.
index 9844d67..8b69671 100755 (executable)
@@ -57,7 +57,7 @@ public class CallFeaturesSetting extends PreferenceActivity
         "com.android.phone.CallFeaturesSetting.ADD_VOICEMAIL";
 
     // debug data
-    private static final String LOG_TAG = "call features settings";
+    private static final String LOG_TAG = "CallFeaturesSetting";
     private static final boolean DBG = false;
 
     // string contants
index 1c674ec..d844a3b 100755 (executable)
@@ -512,8 +512,8 @@ public class CallNotifier extends Handler
     }
 
     void updateCallNotifierRegistrationsAfterRadioTechnologyChange() {
-        if(DBG) Log.d(LOG_TAG, "updateCallNotifierRegistrationsAfterRadioTechnologyChange...");
-        //Unregister all events from the old obsolete phone
+        if (DBG) Log.d(LOG_TAG, "updateCallNotifierRegistrationsAfterRadioTechnologyChange...");
+        // Unregister all events from the old obsolete phone
         mPhone.unregisterForNewRingingConnection(this);
         mPhone.unregisterForPhoneStateChanged(this);
         mPhone.unregisterForDisconnect(this);
@@ -523,12 +523,12 @@ public class CallNotifier extends Handler
         mPhone.unregisterForDisplayInfo(this);
         mPhone.unregisterForSignalInfo(this);
 
-        //Release the ToneGenerator used for playing SignalInfo and CallWaiting
+        // Release the ToneGenerator used for playing SignalInfo and CallWaiting
         if (mSignalInfoToneGenerator != null) {
             mSignalInfoToneGenerator.release();
         }
 
-        //Register all events new to the new active phone
+        // Register all events new to the new active phone
         mPhone.registerForNewRingingConnection(this, PHONE_NEW_RINGING_CONNECTION, null);
         mPhone.registerForPhoneStateChanged(this, PHONE_STATE_CHANGED, null);
         mPhone.registerForDisconnect(this, PHONE_DISCONNECT, null);
index 300b406..88e7fbc 100755 (executable)
@@ -888,6 +888,15 @@ public class InCallScreen extends Activity
         mRegisteredForPhoneStates = false;
     }
 
+    /* package */ void updateAfterRadioTechnologyChange() {
+        if (DBG) Log.d(LOG_TAG, "updateAfterRadioTechnologyChange()...");
+        // Unregister for all events from the old obsolete phone
+        unregisterForPhoneStates();
+
+        // (Re)register for all events relevant to the new active phone
+        registerForPhoneStates();
+    }
+
     @Override
     protected void onNewIntent(Intent intent) {
         if (DBG) log("onNewIntent: intent=" + intent);
@@ -996,7 +1005,6 @@ public class InCallScreen extends Activity
         mCallCard = (CallCard) callCardLayout.findViewById(R.id.callCard);
         if (VDBG) log("  - mCallCard = " + mCallCard);
         mCallCard.setInCallScreenInstance(this);
-        mCallCard.reset();
 
         // Menu Button hint
         mMenuButtonHint = (TextView) findViewById(R.id.menuButtonHint);
index 9be35db..67d719b 100644 (file)
@@ -801,7 +801,6 @@ public class NotificationMgr implements CallerInfoAsyncQuery.OnQueryCompleteList
             }
 
             if (mPhone.getPhoneName().equals("CDMA")) {
-                String finalTitle;
                 int vmCount = mPhone.getVoiceMessageCount();
                 String titleFormat = mContext.getString(R.string.notification_voicemail_title_count);
                 notificationTitle = String.format(titleFormat, vmCount);
index 61d3ded..5591da2 100755 (executable)
@@ -922,18 +922,21 @@ public class PhoneApp extends Application {
     }
 
     private void initForNewRadioTechnology() {
-        if(DBG) Log.d(LOG_TAG, "initForNewRadioTechnology...");
+        if (DBG) Log.d(LOG_TAG, "initForNewRadioTechnology...");
 
         ringer.updateRingerContextAfterRadioTechnologyChange(this.phone);
         notifier.updateCallNotifierRegistrationsAfterRadioTechnologyChange();
-        if(mBtHandsfree != null) {
+        if (mBtHandsfree != null) {
             mBtHandsfree.updateBtHandsfreeAfterRadioTechnologyChange();
         }
+        if (mInCallScreen != null) {
+            mInCallScreen.updateAfterRadioTechnologyChange();
+        }
 
-        //Update registration for ICC status after radio technology change
+        // Update registration for ICC status after radio technology change
         IccCard sim = phone.getIccCard();
         if (sim != null) {
-            if(DBG) Log.d(LOG_TAG, "Update registration for ICC status...");
+            if (DBG) Log.d(LOG_TAG, "Update registration for ICC status...");
 
             //Register all events new to the new active phone
             sim.registerForAbsent(mHandler, EVENT_SIM_ABSENT, null);
@@ -1073,10 +1076,10 @@ public class PhoneApp extends Application {
                 if (VDBG) Log.d(LOG_TAG, "==> new state: " + mBluetoothHeadsetAudioState);
                 updateBluetoothIndication(true);  // Also update any visible UI if necessary
             } else if (action.equals(TelephonyIntents.ACTION_ANY_DATA_CONNECTION_STATE_CHANGED)) {
-                // if (DBG) Log.d(LOG_TAG, "mReceiver: ACTION_ANY_DATA_CONNECTION_STATE_CHANGED");
-                // if (DBG) Log.d(LOG_TAG, "- state: " + intent.getStringExtra(Phone.STATE_KEY));
-                // if (DBG) Log.d(LOG_TAG, "- reason: "
-                //                + intent.getStringExtra(Phone.STATE_CHANGE_REASON_KEY));
+                if (VDBG) Log.d(LOG_TAG, "mReceiver: ACTION_ANY_DATA_CONNECTION_STATE_CHANGED");
+                if (VDBG) Log.d(LOG_TAG, "- state: " + intent.getStringExtra(Phone.STATE_KEY));
+                if (VDBG) Log.d(LOG_TAG, "- reason: "
+                                + intent.getStringExtra(Phone.STATE_CHANGE_REASON_KEY));
 
                 // The "data disconnected due to roaming" notification is
                 // visible if you've lost data connectivity because you're
@@ -1185,7 +1188,7 @@ public class PhoneApp extends Application {
             hasService = false;
         }
 
-        if(ss.getRadioTechnology() == ServiceState.RADIO_TECHNOLOGY_1xRTT
+        if (ss.getRadioTechnology() == ServiceState.RADIO_TECHNOLOGY_1xRTT
                 || ss.getRadioTechnology() == ServiceState.RADIO_TECHNOLOGY_EVDO_0
                 || ss.getRadioTechnology() == ServiceState.RADIO_TECHNOLOGY_EVDO_A
                 || ss.getRadioTechnology() == ServiceState.RADIO_TECHNOLOGY_IS95A
index 7e3ab08..7268880 100755 (executable)
@@ -1130,52 +1130,6 @@ public class PhoneUtils {
     }
 
     /**
-     * Returns a single "name" for the specified Connection.
-     * This may be the caller name, the phone number, or a generic "unknown"
-     * string, depending on what info is available.
-     *
-     * NOTE: This API should be avoided, with preference given to the
-     * asynchronous startGetCallerInfo API, used in conjunction with
-     * getCompactNameFromCallerInfo().
-     */
-    static String getCompactName(Context context, Connection conn) {
-        CallerInfo info = getCallerInfo(context, conn);
-        if (DBG) log("getCompactName: info = " + info);
-
-        String compactName = null;
-        if (info != null) {
-            compactName = info.name;
-            if ((compactName == null) || (TextUtils.isEmpty(compactName))) {
-                compactName = info.phoneNumber;
-            }
-        }
-        // TODO: figure out UNKNOWN, PRIVATE numbers?
-        if ((compactName == null) || (TextUtils.isEmpty(compactName))) {
-            compactName = context.getString(R.string.unknown);
-        }
-        return compactName;
-    }
-
-    /**
-     * Returns a single "name" for the specified Call.
-     * If the call only has a single connection, this is
-     * just like calling getCompactName() on that connection.
-     * But if this call has more than one connection,
-     * return a generic string like "Conference call".
-     *
-     * NOTE: This API should be avoided, with preference given to the
-     * asynchronous startGetCallerInfo API, used in conjunction with
-     * getCompactNameFromCallerInfo().
-     */
-    static String getCompactName(Context context, Call call) {
-        if (isConferenceCall(call)) {
-            return context.getString(R.string.confCall);
-        }
-        Connection conn = call.getEarliestConnection();  // may be null
-        return getCompactName(context, conn);  // OK if conn is null
-    }
-
-    /**
      * Returns a single "name" for the specified given a CallerInfo object.
      * If the name is null, return defaultString as the default value, usually
      * context.getString(R.string.unknown).