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

Commit 59ebc70c authored by Pranav Madapurmath's avatar Pranav Madapurmath
Browse files

Remove additional post from CallSequencingStats 

There are two invocations to Handler#post which is adding unecessary
delay when logging the CallSequencingStats metrics. The invocation in
CallSequencingStats#log is redundant and not needed. This is causing
test flakiness in aosp cf builds.

Bug: 417789369
Flag: EXEMPT test fix
Test: atest TelecomPulledAtomTest
Change-Id: Ib53722672b09d07fa345795444b15a1356f275b6
parent 2d16482b

src/com/android/server/telecom/metrics/CallSequencingStats.java

+4 −6
Original line number Diff line number Diff line
@@ -136,12 +136,10 @@ public class CallSequencingStats extends TelecomPulledAtom { 


    @VisibleForTesting(otherwise = VisibleForTesting.PROTECTED)

    public void log(CallSequencingStatsKey key, int duration) {

        post(() -> {

        CallSequencingStatsData data = mCallSequencingStatsMap

                .computeIfAbsent(key, k -> new CallSequencingStatsData(0, 0));

        data.add(duration);

        onAggregate();

        });

    }



    public void onCallEnd(Call call) {