diff options
author | Girish <girishshetty@google.com> | 2024-02-22 20:30:13 +0000 |
---|---|---|
committer | Girish <girishshetty@google.com> | 2024-04-18 16:55:10 +0000 |
commit | 286204445d237fa2bc1bbe17d6ecb3ce177c4a30 (patch) | |
tree | 5592fbb870724e68cb700f0acc82fcd3d599d122 | |
parent | bd98dc515dcf63b0f65f1a7fc7510208944589a7 (diff) | |
download | av-286204445d237fa2bc1bbe17d6ecb3ce177c4a30.tar.gz |
libstagefright: record failed codec initialization
Bug: 335662059
Test: atest android.media.misc.cts.ResourceManagerTest
atest android.media.misc.cts.ResourceManagerMultiTest
/data/nativetest64/ResourceManagerService_test/ResourceManagerService_test
/data/nativetest64/ResourceObserverService_test/ResourceObserverService_test
Change-Id: I6a37e397c3e6695306c2e496d89f3c1dcee8d0f6
-rw-r--r-- | media/libstagefright/MediaCodec.cpp | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/media/libstagefright/MediaCodec.cpp b/media/libstagefright/MediaCodec.cpp index 4e378d43a2..1a479acd91 100644 --- a/media/libstagefright/MediaCodec.cpp +++ b/media/libstagefright/MediaCodec.cpp @@ -3975,6 +3975,15 @@ void MediaCodec::onMessageReceived(const sp<AMessage> &msg) { switch (mState) { case INITIALIZING: { + // Resource error during INITIALIZING state needs to be logged + // through metrics, to be able to track such occurrences. + if (isResourceError(err)) { + mediametrics_setInt32(mMetricsHandle, kCodecError, err); + mediametrics_setCString(mMetricsHandle, kCodecErrorState, + stateString(mState).c_str()); + flushMediametrics(); + initMediametrics(); + } setState(UNINITIALIZED); break; } |