Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 36f94329 authored by Tyler Gunn's avatar Tyler Gunn
Browse files

Add telecom event for skip ringing. 

Adding telecom event when ringing is skipped, along with the reason why
it was skipped.

Test: Manual
Bug: 112892710
Change-Id: Ied602e0840eaef6ef1449729fb2b2efe00d12048
parent 12ea9653

src/com/android/server/telecom/Ringer.java

+5 −3
Original line number Diff line number Diff line
@@ -167,7 +167,7 @@ public class Ringer { 


        if (endEarly) {

            if (letDialerHandleRinging) {

                Log.addEvent(foregroundCall, LogUtils.Events.SKIP_RINGING);

                Log.addEvent(foregroundCall, LogUtils.Events.SKIP_RINGING, "Dialer handles");

            }

            Log.i(this, "Ending early -- isTheaterModeOn=%s, letDialerHandleRinging=%s, " +

                    "isSelfManaged=%s, hasExternalRinger=%s", isTheaterModeOn,
@@ -188,9 +188,11 @@ public class Ringer { 
            mRingtonePlayer.play(mRingtoneFactory, foregroundCall);

            effect = getVibrationEffectForCall(mRingtoneFactory, foregroundCall);

        } else {

            Log.i(this, "startRinging: skipping because ringer would not be audible. " +

            String reason = String.format(

                    "isVolumeOverZero=%s, shouldRingForContact=%s, isRingtonePresent=%s",

                    isVolumeOverZero, shouldRingForContact, isRingtonePresent);

            Log.i(this, "startRinging: skipping because ringer would not be audible. " + reason);

            Log.addEvent(foregroundCall, LogUtils.Events.SKIP_RINGING, "Inaudible: " + reason);

            effect = mDefaultVibrationEffect;

        }
@@ -224,7 +226,7 @@ public class Ringer { 
        }



        if (mInCallController.doesConnectedDialerSupportRinging()) {

            Log.addEvent(call, LogUtils.Events.SKIP_RINGING);

            Log.addEvent(call, LogUtils.Events.SKIP_RINGING, "Dialer handles");

            return;

        }