Merge "Revert "Un-obscure the Telephony phone account ID."" into tm-dev

This commit is contained in:
Philip Quinn
2022-05-03 07:14:08 +00:00
committed by Android (Google) Code Review

View File

@@ -46,14 +46,6 @@ import java.util.Objects;
* See {@link PhoneAccount}, {@link TelecomManager}. * See {@link PhoneAccount}, {@link TelecomManager}.
*/ */
public final class PhoneAccountHandle implements Parcelable { public final class PhoneAccountHandle implements Parcelable {
/**
* Expected component name of Telephony phone accounts; ONLY used to determine if we should log
* the phone account handle ID.
*/
private static final ComponentName TELEPHONY_COMPONENT_NAME =
new ComponentName("com.android.phone",
"com.android.services.telephony.TelephonyConnectionService");
@UnsupportedAppUsage(maxTargetSdk = Build.VERSION_CODES.P, trackingBug = 127403196) @UnsupportedAppUsage(maxTargetSdk = Build.VERSION_CODES.P, trackingBug = 127403196)
private final ComponentName mComponentName; private final ComponentName mComponentName;
@UnsupportedAppUsage(maxTargetSdk = Build.VERSION_CODES.P, trackingBug = 115609023) @UnsupportedAppUsage(maxTargetSdk = Build.VERSION_CODES.P, trackingBug = 115609023)
@@ -144,23 +136,14 @@ public final class PhoneAccountHandle implements Parcelable {
@Override @Override
public String toString() { public String toString() {
StringBuilder sb = new StringBuilder()
.append(mComponentName)
.append(", ");
if (mComponentName.equals(TELEPHONY_COMPONENT_NAME)) {
// Telephony phone account handles are now keyed by subscription id which is not
// sensitive.
sb.append(mId);
} else {
// Note: Log.pii called for mId as it can contain personally identifying phone account // Note: Log.pii called for mId as it can contain personally identifying phone account
// information such as SIP account IDs. // information such as SIP account IDs.
sb.append(Log.pii(mId)); return new StringBuilder().append(mComponentName)
} .append(", ")
sb.append(", "); .append(Log.pii(mId))
sb.append(mUserHandle); .append(", ")
.append(mUserHandle)
return sb.toString(); .toString();
} }
@Override @Override