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

Commit ee54eeff authored by Ray Essick's avatar Ray Essick
Browse files

Nuplayer2 metrics under 'nuplayer' 

We should be logging the nuplayer2 metrics under 'nuplayer', not
'nuplayer2'.  I had made the poor suggestion that they be nuplayer2.

Bug: 72445608
Test: build
parent c01a25e1

media/libmedia/nuplayer2/NuPlayer2Driver.cpp

+2 −2
Original line number Diff line number Diff line
@@ -77,7 +77,7 @@ private: 
};



// key for media statistics

static const char *kKeyPlayer = "nuplayer2";

static const char *kKeyPlayer = "nuplayer";

// attrs for media statistics

static const char *kPlayerVMime = "android.media.mediaplayer.video.mime";

static const char *kPlayerVCodec = "android.media.mediaplayer.video.codec";
@@ -666,7 +666,7 @@ void NuPlayer2Driver::logMetrics(const char *where) { 


        // re-init in case we prepare() and start() again.

        delete mAnalyticsItem ;

        mAnalyticsItem = new MediaAnalyticsItem("nuplayer");

        mAnalyticsItem = new MediaAnalyticsItem(kKeyPlayer);

        if (mAnalyticsItem) {

            mAnalyticsItem->generateSessionID();

            mAnalyticsItem->setUid(mClientUid);