commit 4c5f4f28a02a3fc421e207034773126f11b09f5f
author Meng Wang <mewan@google.com> Wed Oct 23 14:08:43 2019 -0700
committer Meng Wang <mewan@google.com> Thu Oct 24 21:39:33 2019 +0000
tree 144bd3a4316716ebe553ef02c6595cec237231ec
parent 5316bc9c40ad6b596cf21fd4dc329986c0eab81f

Do not use hidden API SystemProperties.set

Use sysprop-as-api instead.

Bug: 141246285
Test: turn on airplane mode, persist.radio.airplane_mode_on becomes 1;
      turn off airplane mode, persist.radio.airplane_mode_on becomes 0.
Change-Id: I7576cbea559ac9d5411f470182f0c0f2f7ce675d

src/com/android/phone/PhoneGlobals.java

diff --git a/src/com/android/phone/PhoneGlobals.java b/src/com/android/phone/PhoneGlobals.java
index d7ba19c..6b44c4f 100644
--- a/src/com/android/phone/PhoneGlobals.java
+++ b/src/com/android/phone/PhoneGlobals.java
@@ -41,6 +41,7 @@
 import android.os.UserManager;
 import android.preference.PreferenceManager;
 import android.provider.Settings;
+import android.sysprop.TelephonyProperties;
 import android.telecom.TelecomManager;
 import android.telephony.AnomalyReporter;
 import android.telephony.CarrierConfigManager;
@@ -546,7 +547,7 @@
         Log.i(LOG_TAG, "Turning radio off - airplane");
         Settings.Global.putInt(context.getContentResolver(), Settings.Global.CELL_ON,
                  PhoneConstants.CELL_OFF_DUE_TO_AIRPLANE_MODE_FLAG);
-        SystemProperties.set("persist.radio.airplane_mode_on", "1");
+        TelephonyProperties.airplane_mode_on(true); // true means int value 1
         Settings.Global.putInt(getContentResolver(), Settings.Global.ENABLE_CELLULAR_ON_BOOT, 0);
         PhoneUtils.setRadioPower(false);
     }
@@ -557,7 +558,7 @@
                 PhoneConstants.CELL_ON_FLAG);
         Settings.Global.putInt(getContentResolver(), Settings.Global.ENABLE_CELLULAR_ON_BOOT,
                 1);
-        SystemProperties.set("persist.radio.airplane_mode_on", "0");
+        TelephonyProperties.airplane_mode_on(false); // false means int value 0
         PhoneUtils.setRadioPower(true);
     }