Commit 298e6139 by olly Committed by Ian Baker

Improve EventLogger metadata logging

1. Fix indentation
2. Fix intention to only log one Metadata block

PiperOrigin-RevId: 434818209
parent 03c02b8b
...@@ -286,19 +286,19 @@ public class EventLogger implements AnalyticsListener { ...@@ -286,19 +286,19 @@ public class EventLogger implements AnalyticsListener {
} }
// TODO: Replace this with an override of onMediaMetadataChanged. // TODO: Replace this with an override of onMediaMetadataChanged.
// Log metadata for at most one of the selected tracks. // Log metadata for at most one of the selected tracks.
for (int groupIndex = 0; groupIndex < trackGroupInfos.size(); groupIndex++) { boolean loggedMetadata = false;
for (int groupIndex = 0; !loggedMetadata && groupIndex < trackGroupInfos.size(); groupIndex++) {
TracksInfo.TrackGroupInfo trackGroupInfo = trackGroupInfos.get(groupIndex); TracksInfo.TrackGroupInfo trackGroupInfo = trackGroupInfos.get(groupIndex);
TrackGroup trackGroup = trackGroupInfo.getTrackGroup(); TrackGroup trackGroup = trackGroupInfo.getTrackGroup();
for (int trackIndex = 0; trackIndex < trackGroup.length; trackIndex++) { for (int trackIndex = 0; !loggedMetadata && trackIndex < trackGroup.length; trackIndex++) {
if (!trackGroupInfo.isTrackSelected(trackIndex)) { if (trackGroupInfo.isTrackSelected(trackIndex)) {
continue; @Nullable Metadata metadata = trackGroup.getFormat(trackIndex).metadata;
} if (metadata != null && metadata.length() > 0) {
@Nullable Metadata metadata = trackGroup.getFormat(trackIndex).metadata; logd(" Metadata [");
if (metadata != null) { printMetadata(metadata, " ");
logd(" Metadata ["); logd(" ]");
printMetadata(metadata, " "); loggedMetadata = true;
logd(" ]"); }
break;
} }
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment