Merge "Fix bug in enabling/disabling SMS cell broadcast activation." into ics-mr0

Author: Jake Hamby

telephony/java/com/android/internal/telephony/IntRangeManager.java

@@ -542,6 +542,14 @@ private boolean tryAddSingleRange(int startId, int endId, boolean selected) {
         return finishUpdate();
     }
 
+    /**
+     * Returns whether the list of ranges is completely empty.
+     * @return true if there are no enabled ranges
+     */
+    public boolean isEmpty() {
+        return mRanges.isEmpty();
+    }
+
     /**
      * Called when the list of enabled ranges has changed. This will be
      * followed by zero or more calls to {@link #addRange} followed by

telephony/java/com/android/internal/telephony/RIL.java

@@ -1928,7 +1928,7 @@ public void setGsmBroadcastConfig(SmsBroadcastConfigInfo[] config, Message respo
 
         if (RILJ_LOGD) {
             riljLog(rr.serialString() + "> " + requestToString(rr.mRequest)
-                    + " with " + numOfConfig + "configs : ");
+                    + " with " + numOfConfig + " configs : ");
             for (int i = 0; i < numOfConfig; i++) {
                 riljLog(config[i].toString());
             }

telephony/java/com/android/internal/telephony/gsm/SimSmsInterfaceManager.java

@@ -246,6 +246,8 @@ public boolean enableCellBroadcastRange(int startMessageId, int endMessageId) {
             log("Added cell broadcast subscription for MID range " + startMessageId
                     + " to " + endMessageId + " from client " + client);
 
+        setCellBroadcastActivation(!mCellBroadcastRangeManager.isEmpty());
+
         return true;
     }
 
@@ -271,6 +273,8 @@ public boolean disableCellBroadcastRange(int startMessageId, int endMessageId) {
             log("Removed cell broadcast subscription for MID range " + startMessageId
                     + " to " + endMessageId + " from client " + client);
 
+        setCellBroadcastActivation(!mCellBroadcastRangeManager.isEmpty());
+
         return true;
     }
 
@@ -301,14 +305,15 @@ protected void addRange(int startId, int endId, boolean selected) {
         /**
          * Called to indicate the end of a range update started by the
          * previous call to {@link #startUpdate}.
+         * @return true if successful, false otherwise
          */
         protected boolean finishUpdate() {
             if (mConfigList.isEmpty()) {
-                return setCellBroadcastActivation(false);
+                return true;
             } else {
                 SmsBroadcastConfigInfo[] configs =
                         mConfigList.toArray(new SmsBroadcastConfigInfo[mConfigList.size()]);
-                return setCellBroadcastConfig(configs) && setCellBroadcastActivation(true);
+                return setCellBroadcastConfig(configs);
             }
         }
     }