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

Commit 7d234f7c authored by Dichen Zhang's avatar Dichen Zhang
Browse files

Fixes for MediaRecorder log-session-id 

1. Fixed a typo
2. print log-session-id in dumpsys

Test: make, flash and dumpsys
Bug: 188459272
Change-Id: I145f28c0d6ffdaeb87623f3c70bf62d20d8835d9
parent b806407d

services/mediametrics/statsd_recorder.cpp

+3 −4
Original line number Diff line number Diff line
@@ -57,8 +57,8 @@ bool statsd_recorder(const std::shared_ptr<const mediametrics::Item>& item, 


    // string kRecorderLogSessionId = "android.media.mediarecorder.log-session-id";

    std::string log_session_id;

    if (item->getString("android.media.mediarecorder.log_session_id", &log_session_id)) {

        metrics_proto.set_log_session_id(std::move(log_session_id));

    if (item->getString("android.media.mediarecorder.log-session-id", &log_session_id)) {

        metrics_proto.set_log_session_id(log_session_id);

    }

    // string kRecorderAudioMime = "android.media.mediarecorder.audio.mime";

    std::string audio_mime;
@@ -214,8 +214,7 @@ bool statsd_recorder(const std::shared_ptr<const mediametrics::Item>& item, 
            << " video_bitrate:" << video_bitrate



            << " iframe_interval:" << iframe_interval

            // TODO Recorder - add log_session_id

            // << " log_session_id:" << log_session_id

            << " log_session_id:" << log_session_id

            << " }";

    statsdLog->log(android::util::MEDIAMETRICS_RECORDER_REPORTED, log.str());

    return true;