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

Commit 6aee343c authored by Harry Cutts's avatar Harry Cutts
Browse files

TouchpadInputMapper: remove usage atom log lines 

Change Ifc06bdee240471692e8b3c4fc9b67a99d10d203d fixed the flakiness
that these logs were added to diagnose, so we don't need them anymore.

This is a pure revert of changes
I23346b4904f83abb6cc5a07b687bb0a9802c6bfa and
Ic062a5fe5db905c4691147a1267600981b9017e8.

Bug: 324047709
Test: m checkinput
Test: atest 'android.input.cts.hostside.InputAtomsTest#testTouchpadUsageAtom_FingerAndPalmCounts'
Flag: NONE trivial log removal / pure revert
Change-Id: I3114f7444fa40efd8c4ad080e28bbf11f00d1534
parent 278e699b

services/inputflinger/reader/mapper/TouchpadInputMapper.cpp

+0 −3
Original line number Diff line number Diff line
@@ -183,7 +183,6 @@ private: 
    static AStatsManager_PullAtomCallbackReturn pullAtomCallback(int32_t atomTag,

                                                                 AStatsEventList* outEventList,

                                                                 void* cookie) {

        ALOGI("Received pull request for touchpad usage atom");

        LOG_ALWAYS_FATAL_IF(atomTag != android::util::TOUCHPAD_USAGE);

        MetricsAccumulator& accumulator = MetricsAccumulator::getInstance();

        accumulator.produceAtomsAndReset(*outEventList);
@@ -191,14 +190,12 @@ private: 
    }



    void produceAtomsAndReset(AStatsEventList& outEventList) {

        ALOGI("Acquiring lock for touchpad usage metrics...");

        std::scoped_lock lock(mLock);

        produceAtomsLocked(outEventList);

        resetCountersLocked();

    }



    void produceAtomsLocked(AStatsEventList& outEventList) const REQUIRES(mLock) {

        ALOGI("Producing touchpad usage atoms for %zu counters", mCounters.size());

        for (auto& [id, counters] : mCounters) {

            auto [busId, vendorId, productId, versionId] = id;

            addAStatsEvent(&outEventList, android::util::TOUCHPAD_USAGE, vendorId, productId,