commit ca7d4c44d09eccfa4187f668dc450adfda787483
author Meng Wang <mewan@google.com> Thu Jun 30 22:05:24 2016 -0700
committer Meng Wang <mewan@google.com> Fri Jul 08 13:25:20 2016 -0700
tree 039ae0bdab1a32b25e19bdf35e2842c056705257
parent e4344cd61ef3b282c35864eb10e681f1f3caa1da

ImsManager: refine logs for turnOnIms() and turnOffIms()

Bug: 29875759
Change-Id: Id1a88976dfb8313aab297132ae33a60d1972e475

src/java/com/android/ims/ImsManager.java

diff --git a/src/java/com/android/ims/ImsManager.java b/src/java/com/android/ims/ImsManager.java
index f1b2f6e..dbf9fed 100644
--- a/src/java/com/android/ims/ImsManager.java
+++ b/src/java/com/android/ims/ImsManager.java
@@ -371,6 +371,7 @@
                         imsManager.mImsConfigListener);
 
                 if (enabled) {
+                    log("setVtSetting() : turnOnIms");
                     imsManager.turnOnIms();
                 } else if (isTurnOffImsAllowedByPlatform(context)
                         && (!isVolteEnabledByPlatform(context)
@@ -379,7 +380,7 @@
                     imsManager.turnOffIms();
                 }
             } catch (ImsException e) {
-                loge("setVtSetting(): " + e);
+                loge("setVtSetting(): ", e);
             }
         }
     }
@@ -429,6 +430,7 @@
                         imsManager.mImsConfigListener);
 
                 if (enabled) {
+                    log("setWfcSetting() : turnOnIms");
                     imsManager.turnOnIms();
                 } else if (isTurnOffImsAllowedByPlatform(context)
                         && (!isVolteEnabledByPlatform(context)
@@ -442,7 +444,7 @@
                         ? getWfcMode(context)
                         : ImsConfig.WfcModeFeatureValueConstants.CELLULAR_PREFERRED);
             } catch (ImsException e) {
-                loge("setWfcSetting(): " + e);
+                loge("setWfcSetting(): ", e);
             }
         }
     }
@@ -635,7 +637,7 @@
 
                 }
             } catch (ImsException ie) {
-                Rlog.e(TAG, "AsyncUpdateProvisionedValues error: " + ie);
+                Rlog.e(TAG, "AsyncUpdateProvisionedValues error: ", ie);
             }
 
             return null;
@@ -690,15 +692,17 @@
                     // Also, if turning off is not allowed for current carrier,
                     // we need to turn IMS on because it might be turned off before
                     // phone switched to current carrier.
+                    log("updateImsServiceConfig: turnOnIms");
                     imsManager.turnOnIms();
                 } else {
                     // Turn off IMS if it is not used AND turning off is allowed for carrier.
+                    log("updateImsServiceConfig: turnOffIms");
                     imsManager.turnOffIms();
                 }
 
                 imsManager.mConfigUpdated = true;
             } catch (ImsException e) {
-                loge("updateImsServiceConfig: " + e);
+                loge("updateImsServiceConfig: ", e);
                 imsManager.mConfigUpdated = false;
             }
         }
@@ -1357,9 +1361,10 @@
                 }
             }
         } catch (ImsException e) {
-            log("setAdvanced4GMode() : " + e);
+            loge("setAdvanced4GMode() : ", e);
         }
         if (turnOn) {
+            log("setAdvanced4GMode() : turnOnIms");
             turnOnIms();
         } else if (isImsTurnOffAllowed()) {
             log("setAdvanced4GMode() : imsServiceAllowTurnOff -> turnOffIms");